On 05/24/2011 10:25 AM, Jim Meyering wrote: > Another coverity-spotted bug. > Eric, ok to push? > >>From 6dc42e2d25df9c84b335062bad9beb0a7319647b Mon Sep 17 00:00:00 2001 > From: Jim Meyering <meyer...@redhat.com> > Date: Tue, 24 May 2011 18:24:24 +0200 > Subject: [PATCH] utimensat: do not reference an out-of-scope buffer > > Otherwise, with __linux__ defined, "times" would point to a buffer, "ts" > declared in an inner scope, yet "times" would be dereferenced outside > the scope in which "ts" was valid. > * lib/utimensat.c (rpl_utimensat) [__linux__]: Move the declaration > of ts[2] "out/up", so that the use of aliased "times" (via "times = ts;") > does not end up referencing an out-of-scope "ts"
Gotta love coverity for catching this! Please push. -- Eric Blake ebl...@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
