[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-trivial] [PATCH 01/13] qemu-socket: zero-initiali
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [Qemu-trivial] [PATCH 01/13] qemu-socket: zero-initialize SocketAddress |
Date: |
Mon, 24 Jun 2013 08:13:54 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130513 Thunderbird/17.0.6 |
On 06/21/13 20:15, Michael Tokarev wrote:
> 21.06.2013 14:38, Gerd Hoffmann wrote:
>> Signed-off-by: Gerd Hoffmann <address@hidden>
>> ---
>> util/qemu-sockets.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
>> index fdd8dc4..364bd8c 100644
>> --- a/util/qemu-sockets.c
>> +++ b/util/qemu-sockets.c
>> @@ -855,7 +855,7 @@ SocketAddress *socket_parse(const char *str, Error
>> **errp)
>> {
>> SocketAddress *addr = NULL;
>>
>> - addr = g_new(SocketAddress, 1);
>> + addr = g_new0(SocketAddress, 1);
>
> While at it we can remove the =NULL assignment too, guess, someting like this:
>
> --- a/util/qemu-sockets.c
> +++ b/util/qemu-sockets.c
> @@ -848,9 +848,7 @@ int unix_nonblocking_connect(const char *path,
>
> SocketAddress *socket_parse(const char *str, Error **errp)
> {
> - SocketAddress *addr = NULL;
> -
> - addr = g_new(SocketAddress, 1);
> + SocketAddress *addr = addr = g_new0(SocketAddress, 1);
> if (strstart(str, "unix:", NULL)) {
> if (str[5] == '\0') {
> error_setg(errp, "invalid Unix socket address");
>
> Is that okay with you? :)
Yes.
> And not that the original issue is a big issue really, because
> in each case each relevant field is initialized.
Except in the error cases.
cheers,
Gerd
- [Qemu-devel] [PATCH 00/13] A bunch of little socket + chardev fixes, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 01/13] qemu-socket: zero-initialize SocketAddress, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 04/13] qemu-char: check optional fields using has_*, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 03/13] qemu-socket: catch monitor_get_fd failures, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 02/13] qemu-socket: drop pointless allocation, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 06/13] qemu-char: print notification to stderr, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 05/13] qemu-char: use more specific error_setg_* variants, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 07/13] qemu-char: fix documentation for telnet+wait socket flags, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 09/13] qemu-char: use ChardevBackendKind in in CharDriver, Gerd Hoffmann, 2013/06/21
- [Qemu-devel] [PATCH 11/13] qemu-char: add -chardev mux support, Gerd Hoffmann, 2013/06/21