117770 - NPN_GetURLNotify and NPN_PostURLNotify do not fire notification if notifydata is null

Status: VERIFIED DUPLICATE of bug 127259

(Core Graveyard :: Plug-ins, defect)

Description

[av (gone)]

Those API calls take a pointer to a custom data (notifydata) which is supposed 
to be passed back to the plugin when notification comes, so that the plugin can 
distinguish one notification from another. For some reason before firing the 
notification we check this |notifydata| for null:

  if (callbacks->urlnotify != NULL && mNotifyData != nsnull &&
      strnicmp(mNPStream.url, macromediaurl, strlen(macromediaurl)) != 0 )
  {
    ...
    NS_TRY_SAFE_CALL_VOID(CallNPP_URLNotifyProc(callbacks->urlnotify,
                                                npp,
                                                mNPStream.url,
                                                nsPluginReason_Done,
                                                mNotifyData), lib);
    ...
  }
http://lxr.mozilla.org/seamonkey/source/modules/plugin/base/src/ns4xPluginInstan
ce.cpp#164

The Tester plugin easily shows the problem and the difference with NS4x.

Comment 1

[Peter Lubczynski]

nominating nsbeta1, mozilla.10

Comment 2

[Peter Lubczynski]

*** This bug has been marked as a duplicate of 127259 ***

Comment 3

[shrirang khanzode]

mass duplicate verifications . For filtering purposes, pls use keywd
"massdupverification"