[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2] migration: free 'saddr' since be no longer used
From: |
Zongmin Zhou |
Subject: |
[PATCH v2] migration: free 'saddr' since be no longer used |
Date: |
Thu, 16 Nov 2023 14:34:48 +0800 |
Since socket_parse() will allocate memory for 'saddr',and its value
will pass to 'addr' that allocated by migrate_uri_parse(),
then 'saddr' will no longer used,need to free.
But due to 'saddr->u' is shallow copying the contents of the union,
the members of this union containing allocated strings,and will be used after
that.
So just free 'saddr' itself without doing a deep free on the contents of the
SocketAddress.
Fixes: 72a8192e225c ("migration: convert migration 'uri' into 'MigrateAddress'")
Signed-off-by: Zongmin Zhou<zhouzongmin@kylinos.cn>
---
migration/migration.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/migration/migration.c b/migration/migration.c
index 28a34c9068..9bdbcdaf49 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -493,6 +493,7 @@ bool migrate_uri_parse(const char *uri, MigrationChannel
**channel,
}
addr->u.socket.type = saddr->type;
addr->u.socket.u = saddr->u;
+ g_free(saddr);
} else if (strstart(uri, "file:", NULL)) {
addr->transport = MIGRATION_ADDRESS_TYPE_FILE;
addr->u.file.filename = g_strdup(uri + strlen("file:"));
--
2.34.1
- [PATCH] migration: free 'saddr' since be no longer used, Zongmin Zhou, 2023/11/14
- Re: [PATCH] migration: free 'saddr' since be no longer used, Daniel P . Berrangé, 2023/11/15
- Re: [PATCH] migration: free 'saddr' since be no longer used, Peter Xu, 2023/11/15
- [PATCH v2] migration: free 'saddr' since be no longer used,
Zongmin Zhou <=
- Re: [PATCH v2] migration: free 'saddr' since be no longer used, Juan Quintela, 2023/11/16
- Re: [PATCH v2] migration: free 'saddr' since be no longer used, Zongmin Zhou, 2023/11/16
- Re: [PATCH v2] migration: free 'saddr' since be no longer used, Peter Xu, 2023/11/17
- [PATCH v3] migration: free 'saddr' since be no longer used, Zongmin Zhou, 2023/11/19
- Re: [PATCH v3] migration: free 'saddr' since be no longer used, Daniel P . Berrangé, 2023/11/20
- Re: [PATCH v3] migration: free 'saddr' since be no longer used, Peter Xu, 2023/11/20
- Re: [PATCH v3] migration: free 'saddr' since be no longer used, Zongmin Zhou, 2023/11/28
- Re: [PATCH v3] migration: free 'saddr' since be no longer used, Peter Xu, 2023/11/29
- Re: [PATCH v3] migration: free 'saddr' since be no longer used, Het Gala, 2023/11/30