564068 - Test plugin should use NPN allocator to create strings

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[[e-mail]]

STRINGZ_TO_NPVARIANT assumes a string allocated with memalloc, not strdup. The test plugin can cheat because it links with MOZCRT19, but real plugins won't.

There's no NPN strdup function, but one could be created like this:

char*
NPN_StrDup(const char* str)
{
  return strcpy((char*)sBrowserFuncs->memalloc(strlen(str) + 1), str);
}

Comment 1

[[e-mail]]

Also the test plugin needs to be compiled with USE_STATIC_LIBS=1 to stop this mistake happening in future.

Comment 2

[[e-mail]]

Attachment: Possible patch

If I only set USE_STATIC_LIBS = 1 then my debug non-jemalloc build aborts the test because it tries to free memory allocated on the wrong heap, and the rest of the patch fixes that. I'm not actually sure that any tinderboxes would test for that, since I can't build with jemalloc.

Comment 3

[[e-mail]]

Pushed changeset aa0171d36805 to mozilla-central.