Red Hat Bugzilla – Bug 53727
plugger crashes when loading this page
Last modified: 2008-05-01 11:38:00 EDT
Description of Problem:
I've got the lastest Roswell installed. While checking mozilla's bugquest
for a mozilla crash on an unrelated problem, I did run across one similar
bug report, at http://bugzilla.mozilla.org/show_bug.cgi?id=97755
I was unable to duplicate the bug at this page because mozilla crashed on
it, but earlier, apparently for a different reason.
The page has some content that "plugger" tries to handle. Plugger crashes
I tried the same page under Netscape, which uses the same plugger library.
Netscape has no problems with it.
Version-Release number of selected component (if applicable):
The bug occurs every time.
Steps to Reproduce:
1. Open mozila or netscape
2. Load the URL:
Mozilla crashes, Netscape doesn't.
Mozilla shouldn't crash, multimedia content should play (some kind of
animation and sound clip play under netscape).
The info on the plugger home page didn't make it look like plugger is well
supported for mozilla. It mentioned that the author had heard that it
should work on the 0.8 series...
After compiling the source RPM with debugger symbols turned on, it looks like
plugger is dying in plugger.c, function NPP_StreamAsFile, on line 1219.
Changing line 1219 from this:
if((!strcmp(THIS->command, "internal:url")) && fname)
if((!THIS->command || !strcmp(THIS->command, "internal:url")) && fname)
fixes a NULL pointer dereference that is causing a segfault. Mozilla is passing
a NULL command string to plugger. With this change, mozilla doesn't crash, and
plugger seems to continue to work when I browse to, and select subsequent files
that cause plugger to run.
I'm not certain if this is the best fix or not - it might be better to
initialize the command pointer to a NULL-terminated string, but I didn't poke
around long enough to make sure I understood how to safely allocate and free
memory without leaks.
I'll send a link to this bug report to the plugger author (Fredrik H|binette:
email@example.com) to see what he thinks about this.
thanks for your patch file. The plugger-3.3-5 in rawhide includes this patch.