fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-02-05 18:10:29 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

settings: fix invalid signature in impl_settings_get_connection_by_uuid()

Browse source 
impl_settings_get_connection_by_uuid() was changed from a synchronous
function to an asynchronous one. Thereby the @out_object_path argument
was forgotten, leaving the function completely broken.

This bug has the potential to crash NetworkManager.

Regression introduced by commit 8ab8990938.

Found running
  $ ./dfuzzer -v -n com.redhat.ifcfgrh1

Related: https://bugzilla.redhat.com/show_bug.cgi?id=1113508

Signed-off-by: Thomas Haller <thaller@redhat.com>
(cherry picked from commit c35d63ddea)
This commit is contained in:
Thomas Haller 2014-06-26 17:49:27 +02:00
parent ce0c8f29c6
commit 3f1c3c4fc8
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/settings/nm-settings.c
View file

@ -93,7 +93,6 @@ static gboolean impl_settings_list_connections (NMSettings *self,
static void impl_settings_get_connection_by_uuid (NMSettings *self,
const char *uuid,
char **out_object_path,
DBusGMethodInvocation *context);
static void impl_settings_add_connection (NMSettings *self,
@ -272,7 +271,6 @@ nm_settings_get_connection_by_uuid (NMSettings *self, const char *uuid)
static void
impl_settings_get_connection_by_uuid (NMSettings *self,
const char *uuid,
char **out_object_path,
DBusGMethodInvocation *context)
{
NMSettingsConnection *connection = NULL;