free()ing a _strdup'd value correctly?

[xwielder]

In this EXAMPLE, am I free()ing this _strdup()'d return value correctly?

char *getText();

void someFunction(void)
{
    char *msg = NULL;
    msg = &*getText());
    
    // This prints out "Hello World"
    printf("%s", msg);
    
    if (msg)
        free (msg);
}

char *getText()
{
    std::string retVal = "Hello World"; 
    return (_strdup(retVal.c_str()));
}

Thanks!

[Duthomhas]

Yes. You don't actually need line 11. Also, lines 5 and 6 should look like

char *msg = getText();

Hope this helps.

[xwielder]

It does help, and I thank you.

[Duthomhas]

Sorry, it's late and my brain is not functioning. I meant to also say that line 11 would be a good idea before line 9. Don't printf( "%s", NULL );

Enjoy!

[xwielder]

char *getText();

void someFunction(void)
{
    char *msg = getText();
    if (msg)
    {
        printf("%s", msg);
        free (msg);
    }
}

char *getText()
{
    std::string retVal = "Hello World"; 
    return (_strdup(retVal.c_str()));
}

Love it. Thanks!