diff options
author | Markus Armbruster <armbru@redhat.com> | 2020-07-14 18:02:00 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2020-07-21 16:23:43 +0200 |
commit | 7a309cc95b862be74711e7a35997daa3f885651a (patch) | |
tree | 1417043015cf7f35de0ebd21ff2d72a05a5b3b52 /scsi/pr-manager-helper.c | |
parent | af3d69058e09bede9900f266a618ed11f76f49f3 (diff) |
qom: Change object_get_canonical_path_component() not to malloc
object_get_canonical_path_component() returns a malloced copy of a
property name on success, null on failure.
19 of its 25 callers immediately free the returned copy.
Change object_get_canonical_path_component() to return the property
name directly. Since modifying the name would be wrong, adjust the
return type to const char *.
Drop the free from the 19 callers become simpler, add the g_strdup()
to the other six.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20200714160202.3121879-4-armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Li Qiang <liq3ea@gmail.com>
Diffstat (limited to 'scsi/pr-manager-helper.c')
-rw-r--r-- | scsi/pr-manager-helper.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/scsi/pr-manager-helper.c b/scsi/pr-manager-helper.c index bf62cbec11..5acccfb4e3 100644 --- a/scsi/pr-manager-helper.c +++ b/scsi/pr-manager-helper.c @@ -42,11 +42,10 @@ typedef struct PRManagerHelper { static void pr_manager_send_status_changed_event(PRManagerHelper *pr_mgr) { - char *id = object_get_canonical_path_component(OBJECT(pr_mgr)); + const char *id = object_get_canonical_path_component(OBJECT(pr_mgr)); if (id) { qapi_event_send_pr_manager_status_changed(id, !!pr_mgr->ioc); - g_free(id); } } |