[vlc-devel] [PATCH 4/4] text/url: fix port handling in vlc_UrlParse

Filip Roséen filip at atch.se
Fri Oct 28 16:22:16 CEST 2016


Hi again,

On 2016-10-28 14:57, Filip Roséen wrote:

> I have attached an update patch where the state of `errno` is set to `0` prior
> to the invocation of `strtoul`, and reset back to whatever state it had before
> (unless we find invalid data) after.

This is somewhat embarrasing, but for some reason my rebase skill was
*MIA* when I generated that patch (and the actual important change was
not included).

Attached is a patch that, definitely, resets `errno` to `0` prior to
calling `strtoul` (in order for the value to be trusted when verifying
the input).

/F
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20161028/7d1a367f/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-text-url-fix-port-handling-in-vlc_UrlParse.patch
Type: text/x-diff
Size: 1821 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20161028/7d1a367f/attachment.patch>


More information about the vlc-devel mailing list