diff options
| author | Arun Menon <armenon@redhat.com> | 2025-09-18 20:53:29 +0530 |
|---|---|---|
| committer | Peter Xu <peterx@redhat.com> | 2025-10-03 09:48:02 -0400 |
| commit | 3f9d6e77b0471c844d5050cc71db0863eb4d1b64 (patch) | |
| tree | e6c698c5c3d35344f00113e1331e6e6ff24cd7cd /migration/qemu-file.c | |
| parent | 8b6dad124ba757ba96a3b4456223e53632371bc8 (diff) | |
| download | focaccia-qemu-3f9d6e77b0471c844d5050cc71db0863eb4d1b64.tar.gz focaccia-qemu-3f9d6e77b0471c844d5050cc71db0863eb4d1b64.zip | |
migration: Update qemu_file_get_return_path() docs and remove dead checks
The documentation of qemu_file_get_return_path() states that it can return NULL on failure. However, a review of the current implementation reveals that it is guaranteed that it will always succeed and will never return NULL. As a result, the NULL checks post calling the function become redundant. This commit updates the documentation for the function and removes all NULL checks throughout the migration code. Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Fabiano Rosas <farosas@suse.de> Signed-off-by: Arun Menon <armenon@redhat.com> Tested-by: Fabiano Rosas <farosas@suse.de> Reviewed-by: Akihiko Odaki <odaki@rsg.ci.i.u-tokyo.ac.jp> Link: https://lore.kernel.org/r/20250918-propagate_tpm_error-v14-12-36f11a6fb9d3@redhat.com Signed-off-by: Peter Xu <peterx@redhat.com>
Diffstat (limited to 'migration/qemu-file.c')
| -rw-r--r-- | migration/qemu-file.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/migration/qemu-file.c b/migration/qemu-file.c index 0f4280df21..0ee0f48a3e 100644 --- a/migration/qemu-file.c +++ b/migration/qemu-file.c @@ -125,7 +125,6 @@ static QEMUFile *qemu_file_new_impl(QIOChannel *ioc, bool is_writable) /* * Result: QEMUFile* for a 'return path' for comms in the opposite direction - * NULL if not available */ QEMUFile *qemu_file_get_return_path(QEMUFile *f) { |