Re: [Xen-devel] [PATCH 1/4] common/vsprintf: Explicitly treat negative lengths as 'unlimited'

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Thu, 2013-11-28 at 16:48 +0000, Andrew Cooper wrote:
> On 28/11/13 16:37, Tim Deegan wrote:
> > The old code relied on implictly casting negative numbers to size_t
> > making a very large limit, which was correct but non-obvious.
> >
> > Coverity CID 1128575
> >
> > Signed-off-by: Tim Deegan <tim@xxxxxxx>
> 
> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> 
> This CID was introduced as a side effect of my %ps/%pS series, which was
> basically code motion for this piece.  The previous code was not exactly
> fantastic.

I wonder if we should therefore take this for 4.4?



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel