Commit 45b00c44 authored by Marc-André Lureau's avatar Marc-André Lureau
Browse files

contrib: remove unnecessary strdup() 



getopt() optarg points to argv memory, no need to dup those values,
fixes small leaks detected by clang-analyzer.

Signed-off-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
parent 5105b1d8

contrib/ivshmem-client/main.c

+1 −1
Original line number Diff line number Diff line
@@ -53,7 +53,7 @@ ivshmem_client_parse_args(IvshmemClientArgs *args, int argc, char *argv[]) 
            break;



        case 'S': /* unix_sock_path */

            args->unix_sock_path = strdup(optarg);

            args->unix_sock_path = optarg;

            break;



        default:

contrib/ivshmem-server/main.c

+3 −3
Original line number Diff line number Diff line
@@ -92,15 +92,15 @@ ivshmem_server_parse_args(IvshmemServerArgs *args, int argc, char *argv[]) 
            break;



        case 'p': /* pid_file */

            args->pid_file = strdup(optarg);

            args->pid_file = optarg;

            break;



        case 'S': /* unix_socket_path */

            args->unix_socket_path = strdup(optarg);

            args->unix_socket_path = optarg;

            break;



        case 'm': /* shm_path */

            args->shm_path = strdup(optarg);

            args->shm_path = optarg;

            break;



        case 'l': /* shm_size */