[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] readline: Fix unchecked strdup() by conve
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] readline: Fix unchecked strdup() by converting to g_strdup() |
Date: |
Wed, 16 Jan 2013 17:03:30 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 01/16/2013 07:36 AM, Markus Armbruster wrote:
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> readline.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/readline.c b/readline.c
>> index a0c9638..d6e04d4 100644
>> --- a/readline.c
>> +++ b/readline.c
>> @@ -247,14 +247,14 @@ static void readline_hist_add(ReadLineState *rs, const
>> char *cmdline)
>> }
>> if (idx == READLINE_MAX_CMDS) {
>> /* Need to get one free slot */
>> - free(rs->history[0]);
>> + g_free(rs->history[0]);
>> memmove(rs->history, &rs->history[1],
>> (READLINE_MAX_CMDS - 1) * sizeof(char *));
>> rs->history[READLINE_MAX_CMDS - 1] = NULL;
>> idx = READLINE_MAX_CMDS - 1;
>> }
>> if (new_entry == NULL)
>> - new_entry = strdup(cmdline);
>> + new_entry = g_strdup(cmdline);
>
> As long as you are converting tabs to spaces, you should also be adding {}.
I style-fix exactly the lines I touch anyway. For what it's worth,
checkpatch is happy.
- [Qemu-devel] [PATCH 00/11] Trivial memory allocation fixes & cleanups, Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 04/11] readline: Fix unchecked strdup() by converting to g_strdup(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 05/11] spice: Fix unchecked strdup() by converting to g_strdup(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 11/11] qemu-ga: Fix unchecked strdup() by converting to g_strdup(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 06/11] virtfs-proxy-helper: Fix unchecked strdup() by conv. to g_strdup(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 02/11] g_strdup(NULL) returns NULL; simplify, Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 03/11] hw/9pfs: Fix unchecked strdup() by converting to g_strdup(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 08/11] qemu-log: Plug trivial memory leak in cpu_set_log_filename(), Markus Armbruster, 2013/01/16
- [Qemu-devel] [PATCH 10/11] qapi: Fix unchecked strdup() by converting to g_strdup(), Markus Armbruster, 2013/01/16