[Libreoffice] va_start without va_end

Caolán McNamara caolanm at redhat.com
Thu Aug 4 01:33:05 PDT 2011


On Wed, 2011-08-03 at 23:56 +0200, Julien Nabet wrote:
> Hello,
> 
> In svl/source/items/itemset.cxx b/svl/source/items/itemset.cxx, I 
> noticed the use of va_start without va_end.
> I read it could create undefined behaviour, so I propose this simple patch.
> 
...
> +        va_end(pArgs);
>       }
>   }
> 
> If it's ok, i can commit and push it on master.

The va_end is tucked away hidden inside InitializeRanges_Impl in
svl/source/items/nranges.cxx

This looks rather ugly, it would be nicer to get the va_start and va_end
closer together, e.g. move the va_end out of InitializeRanges_Impl and
put it close to the two va_starts

C.



More information about the LibreOffice mailing list