Commit d6014bde authored by Daniel Amm's avatar Daniel Amm Committed by Jean-Baptiste Kempf

npapi: fix possible crash

Signed-off-by: Jean-Baptiste Kempf's avatarJean-Baptiste Kempf <jb@videolan.org>
parent f219ed8c
...@@ -12,6 +12,9 @@ ActiveX: ...@@ -12,6 +12,9 @@ ActiveX:
* Allow empty strings for aspectRatio and crop * Allow empty strings for aspectRatio and crop
* Fix initial volume value for volume attribute * Fix initial volume value for volume attribute
NPAPI:
* Fix possible crash in vlc.playlist.add()
Changes between VLC plugins 2.2.2 and 2.2.4: Changes between VLC plugins 2.2.2 and 2.2.4:
-------------------------------------------- --------------------------------------------
......
...@@ -1133,6 +1133,10 @@ void LibvlcPlaylistNPObject::parseOptions(NPObject *obj, int *i_options, ...@@ -1133,6 +1133,10 @@ void LibvlcPlaylistNPObject::parseOptions(NPObject *obj, int *i_options,
NPIdentifier propId = NPN_GetStringIdentifier("length"); NPIdentifier propId = NPN_GetStringIdentifier("length");
if( NPN_GetProperty(_instance, obj, propId, value) ) if( NPN_GetProperty(_instance, obj, propId, value) )
{ {
/* Check if result is valid (because we don't use NPN_HasProperty, the result can be void) */
if( !value.is<int>() )
return;
int count = value; int count = value;
if( count ) if( count )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment