UPSTREAM: console/vtxprintf.c: cast precision to size_t for string length

If no maximum string length is specified, we're intentionally passing a
value of -1 to get the string length so that it's not limited.  This
makes checking tools unhappy, so actively cast it to size_t before
passing it into strlen to show that it's not an accident.

Addresses coverity issue 1129133 - Argument cannot be negative

BUG=None
BRANCH=None
TEST=None

Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/17479
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>

Change-Id: I40f8f2101e170a5c96fcd39c217aa414f4316473
Reviewed-on: https://chromium-review.googlesource.com/415044
Commit-Ready: Aaron Durbin <adurbin@chromium.org>
Tested-by: Furquan Shaikh <furquan@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Martin Roth 2016-11-18 11:35:01 -07:00 committed by chrome-bot
parent c0bf18f799
commit 30d88884c5

View file

@ -228,7 +228,7 @@ repeat:
if (!s)
s = "<NULL>";
len = strnlen(s, precision);
len = strnlen(s, (size_t)precision);
if (!(flags & LEFT))
while (len < field_width--)