[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.