Windows memory leak in v8 wrapper

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Duplicate
    • Priority: Critical - P2
    • None
    • Affects Version/s: 2.3.0
    • Component/s: JavaScript
    • None
    • Windows
    • None
    • 0
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      In v8_db.cpp, the GETNS macro generates an automatic variable-length array variable, except on Windows where we just allocate memory on the heap and never free it:

      #ifdef _WIN32
      #define GETNS char * ns = new char[args[0]->ToString()->Utf8Length()];  args[0]->ToString()->WriteUtf8( ns );
      #else
      #define GETNS char ns[args[0]->ToString()->Utf8Length()];  args[0]->ToString()->WriteUtf8( ns );
      #endif
      

              Assignee:
              Tad Marshall
              Reporter:
              Eric Milkie
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: