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

      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: