fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2025-12-28 11:00:09 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

cli: fix crash on autocompletion

Browse source 
@connections is NULL when doing autocompletion. Fixes the following:

 $ nmcli --complete-args con monitor ""
  help
  id
  uuid
  path
  filename
  ...
  Segmentation fault (core dumped)

Fixes: 4b3297271e ('cli: rework connection handling for multiple results')

https://bugzilla.redhat.com/show_bug.cgi?id=1716948
https://gitlab.freedesktop.org/NetworkManager/NetworkManager/merge_requests/177
This commit is contained in:
Beniamino Galvani 2019-06-05 15:17:08 +02:00
parent 32c6c0cdaf
commit 6a3bb90ad4
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
clients/cli/connections.c
View file

@ -8741,6 +8741,9 @@ do_connection_monitor (NmCli *nmc, int argc, char **argv)
}
}
if (nmc->complete)
return nmc->return_value;
for (i = 0; i < connections->len; i++)
connection_watch (nmc, connections->pdata[i]);
@ -8750,8 +8753,6 @@ do_connection_monitor (NmCli *nmc, int argc, char **argv)
g_signal_connect (nmc->client, NM_CLIENT_CONNECTION_ADDED, G_CALLBACK (connection_added), nmc);
}
if (nmc->complete)
return nmc->return_value;
g_signal_connect (nmc->client, NM_CLIENT_CONNECTION_REMOVED, G_CALLBACK (connection_removed), nmc);
return NMC_RESULT_SUCCESS;