Question

I am using this code inside a class to make a webbrowser control visit a website:

void myClass::visitWeb(const char *url)
{
    WCHAR buffer[MAX_LEN];
    ZeroMemory(buffer, sizeof(buffer));
    MultiByteToWideChar(CP_ACP, MB_ERR_INVALID_CHARS, url, strlen(url), buffer, sizeof(buffer)-1);

    VARIANT vURL;
    vURL.vt = VT_BSTR;
    vURL.bstrVal = SysAllocString(buffer);

    // webbrowser navigate code...

    VariantClear(&vURL);
}

I call visitWeb from another void function that gets called on the handlemessage() for the app.
Do I need to do some memory deallocation here?, I see vURL is being deallocated by VariantClear but should I deallocate memory for buffer?
I’ve been told that in another bool I have in the same app I shouldn’t deallocate anything because everything clear out when the bool return true/false, but what happens on this void?

Answer 1

I think you have some fundamental problems with your understanding of memory management. In this case, no, you don’t need to explicitly free any memory. You didn’t ever call new, so you don’t need to call delete. buffer exists only on the stack, and will vanish when this method returns.