[fedora-virt] [PATCH] add comments suggesting memory-handling improvements
Jim Meyering
jim at meyering.net
Wed Jul 1 16:41:47 UTC 2009
Richard W.M. Jones wrote:
> On Wed, Jul 01, 2009 at 06:21:05PM +0200, Jim Meyering wrote:
>> I've include a patch below. It would have been easier and cleaner *looking*
>> to use functions like xrealloc, xstrdup, etc. that exit upon failure here.
>> But since nothing else seems to exit from this code, I bit the bullet:
>> (this compiles, but "make check" hasn't completed yet)
>
> Yes, I agree. There's not very much that guestfish can do if it runs
> out of memory, and the most sensible thing would be just to exit with
> an error (or abort).
>
>> Hmm... new test failures on F11 (details below)
>> But I'd just pulled latest and rebased, too.
>>
>> >From 77b4a54834cd3d3e6994508104334f501f5d99f1 Mon Sep 17 00:00:00 2001
>> From: Jim Meyering <meyering at redhat.com>
>> Date: Wed, 1 Jul 2009 16:09:33 +0200
>> Subject: [PATCH] fish: handle some out-of-memory conditions
>>
>> * fish/destpaths.c (xalloc_oversized): Define.
>> (complete_dest_paths_generator): Use size_t as type for a few
>> variables, rather than int.
>> Don't deref NULL or undef on failed heap alloc.
>> Don't leak on failed realloc.
>> Detect theoretical overflow when count_strings returns a very
>> large number of strings.
>> Handle asprintf failure.
>> (APPEND_STRS_AND_FREE): Rewrite as do {...}while(0), so that each use
>> can/must be followed by a semicolon. Better for auto-formatters.
>> ---
>> fish/destpaths.c | 91 ++++++++++++++++++++++++++++++++++++-----------------
>> 1 files changed, 62 insertions(+), 29 deletions(-)
>>
>> diff --git a/fish/destpaths.c b/fish/destpaths.c
>> index 6cddafa..90970de 100644
>> --- a/fish/destpaths.c
>> +++ b/fish/destpaths.c
>> @@ -1,5 +1,5 @@
>> /* guestfish - the filesystem interactive shell
>> - * Copyright (C) 2009 Red Hat Inc.
>> + * Copyright (C) 2009 Red Hat Inc.
>
> Strange, this doesn't look like any change?
As I sent it, I accidentally let a write hook remove the trailing blank
on the "- ..." line.
I.e., that change (before being munged) removes the sole
trailing blank in that file.
More information about the virt
mailing list