Re: [Xen-devel] [PATCH 3/8] tools/xenalyze: Handle fstat errors properly

2016-03-03 Thread George Dunlap
On 26/02/16 12:25, Ian Jackson wrote: > George Dunlap writes ("[PATCH 3/8] tools/xenalyze: Handle fstat errors > properly"): >> They're pretty unlikely to fail, but doesn't hurt to check. > ... >> -fstat(fd, &s); >> +if ( fstat(fd, &s) ) { >> +perror("fstat"); >> +free(h);

Re: [Xen-devel] [PATCH 3/8] tools/xenalyze: Handle fstat errors properly

2016-02-26 Thread Ian Jackson
George Dunlap writes ("[PATCH 3/8] tools/xenalyze: Handle fstat errors properly"): > They're pretty unlikely to fail, but doesn't hurt to check. ... > -fstat(fd, &s); > +if ( fstat(fd, &s) ) { > +perror("fstat"); > +free(h); > +h = NULL; > +goto out; > +

[Xen-devel] [PATCH 3/8] tools/xenalyze: Handle fstat errors properly

2016-02-25 Thread George Dunlap
They're pretty unlikely to fail, but doesn't hurt to check. CID 1311502 CID 1311501 Signed-off-by: George Dunlap --- CC: Ian Jackson CC: Wei Liu --- tools/xentrace/mread.c| 9 - tools/xentrace/xenalyze.c | 5 - 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/to