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);
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;
> +
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