[vlc-devel] [PATCH 1/2] mosaic_bridge: fix format-overflow warning

Thomas Guillem thomas at gllm.fr
Thu Sep 5 08:56:43 CEST 2019


On Thu, Sep 5, 2019, at 08:17, Steve Lhomme wrote:
> Isn't it supposed to translate a NULL pointer to "NULL" ?

on Linux: "(nil)" but it's undefined.

> 
> On 2019-09-05 1:35, Alexandre Janniaux wrote:
> > ---
> >   modules/stream_out/mosaic_bridge.c | 5 +++--
> >   1 file changed, 3 insertions(+), 2 deletions(-)
> > 
> > diff --git a/modules/stream_out/mosaic_bridge.c b/modules/stream_out/mosaic_bridge.c
> > index fa5d00c259..d54a3d15c7 100644
> > --- a/modules/stream_out/mosaic_bridge.c
> > +++ b/modules/stream_out/mosaic_bridge.c
> > @@ -386,7 +386,7 @@ static void *Add( sout_stream_t *p_stream, const es_format_t *p_fmt )
> >       };
> > 
> >       psz_chain = var_GetNonEmptyString( p_stream, CFG_PREFIX "vfilter" );
> > -    msg_Dbg( p_stream, "psz_chain: %s", psz_chain );
> > +    msg_Dbg( p_stream, "psz_chain: %s", psz_chain ? psz_chain : "");
> >       if( psz_chain )
> >       {
> >           filter_owner_t owner = {
> > @@ -581,7 +581,8 @@ inline static int video_update_format_decoder( decoder_t *p_dec )
> >       {
> >           // update the filter after the format changed/is known
> >           char *psz_chain = var_GetNonEmptyString( p_owner->p_stream, CFG_PREFIX "vfilter" );
> > -        msg_Dbg( p_owner->p_stream, "update filter: %s", psz_chain );
> > +        msg_Dbg( p_owner->p_stream, "update filter: %s",
> > +                 psz_chain ?  psz_chain : "" );
> >           if( psz_chain )
> >           {
> >               es_format_t fmt;
> > --
> > 2.23.0
> > 
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
> > 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list