<br><br><div class="gmail_quote">On Fri, Jun 24, 2011 at 3:05 AM, Laurent Aimar <span dir="ltr"><<a href="mailto:fenrir@elivagar.org">fenrir@elivagar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi,<br>
<div class="im"><br>
On Fri, Jun 24, 2011 at 02:56:11AM +0530, akash mehrotra wrote:<br>
> Some more minor fixes.<br>
> I hope its correct this time.<br>
<br>
</div>> @@ -51,6 +52,12 @@ static int vlclua_preamp_get( lua_State *L )<br>
>      if( p_input )<br>
>      {<br>
<div class="im">>          aout_instance_t *p_aout = input_GetAout( p_input );<br>
</div>> +        char *psz_af = var_GetNonEmptyString( p_aout, "audio-filter" );<br>
> +        if ( !p_aout || strstr ( psz_af, "equalizer" ) == NULL )<br>
 The p_aout check seems badly placed, var_Get* should not be called on<br>
NULL objects.<br>
 var_GetNonEmptyString will return NULL for empty strings, and then strstr()<br>
will probably segfault.<br>
<br>
(Same for the following ones).<br>
<font color="#888888"><br></font></blockquote><div>Would it be OK this way? <br><br> aout_instance_t *p_aout = input_GetAout( p_input );<br>        if ( !p_aout )<br>        {<br>            vlc_object_release( p_input );<br>
            return 0;<br>        }<br>        char *psz_af = var_GetNonEmptyString( p_aout, "audio-filter" );<br>        if ( strstr ( psz_af, "equalizer" ) == NULL )<br>        {<br>            vlc_object_release( p_aout );<br>
            vlc_object_release( p_input );<br>            return 0;<br>        }<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><font color="#888888">
--<br>
fenrir<br>
</font><div><div></div><div class="h5"><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>