Stefan Hajnoczi <stefa...@gmail.com> writes:
> On Wed, Jan 16, 2013 at 06:32:15PM +0100, Markus Armbruster wrote:
>> Signed-off-by: Markus Armbruster <arm...@redhat.com>
>> Reviewed-by: Eric Blake <ebl...@redhat.com>
>> ---
>> fsdev/virtfs-proxy-helper.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/fsdev/virtfs-proxy-helper.c b/fsdev/virtfs-proxy-helper.c
>> index 6b9afd3..36f6616 100644
>> --- a/fsdev/virtfs-proxy-helper.c
>> +++ b/fsdev/virtfs-proxy-helper.c
>> @@ -1039,7 +1039,7 @@ int main(int argc, char **argv)
>> }
>> switch (c) {
>> case 'p':
>> - rpath = strdup(optarg);
>> + rpath = g_strdup(optarg);
>> break;
>> case 'n':
>> is_daemon = false;
>> @@ -1048,7 +1048,7 @@ int main(int argc, char **argv)
>> sock = atoi(optarg);
>> break;
>> case 's':
>> - sock_name = strdup(optarg);
>> + sock_name = g_strdup(optarg);
>
> rpath and sock_name are leaked. Not important though because they are
> in main().
Yup. Freeing stuff before main returns isn't worth your while or mine
:)
