From 193d1255e6aff1f7aae546cb3a013c39ad8b96b0 Mon Sep 17 00:00:00 2001 From: Bernhard Voelker Date: Fri, 2 Aug 2013 14:22:37 +0200 Subject: [PATCH] xargs: Fix integer overflow in implicit conversion in do_exec. To: findutils-patches@gnu.org * xargs/xargs.c (xargs_do_exec): Change type of r from int to size_t, since that is the type that safe_read actually returns. --- ChangeLog | 6 ++++++ xargs/xargs.c | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 678773e..411eb5d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2013-09-20 Bernhard Voelker + + xargs: Fix integer overflow in implicit conversion in do_exec. + * xargs/xargs.c (xargs_do_exec): Change type of r from int to + size_t, since that is the type that safe_read actually returns. + 2013-09-20 James Youngman Add Bernhard Voelker to AUTHORS. diff --git a/xargs/xargs.c b/xargs/xargs.c index e0b2421..d773bde 100644 --- a/xargs/xargs.c +++ b/xargs/xargs.c @@ -1173,7 +1173,7 @@ xargs_do_exec (struct buildcmd_control *ctl, void *usercontext, int argc, char * pid_t child; int fd[2]; int buf; - int r; + size_t r; (void) ctl; (void) argc; -- 1.7.10.4