[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/3] migration/multifd: Duplicate the fd for the outgoing_
From: |
Peter Xu |
Subject: |
Re: [PATCH v3 2/3] migration/multifd: Duplicate the fd for the outgoing_args |
Date: |
Fri, 15 Mar 2024 07:39:48 -0400 |
On Fri, Mar 15, 2024 at 12:20:39AM -0300, Fabiano Rosas wrote:
> We currently store the file descriptor used during the main outgoing
> channel creation to use it again when creating the multifd
> channels.
>
> Since this fd is used for the first iochannel, there's risk that the
> QIOChannel gets freed and the fd closed while outgoing_args.fd still
> has it available. This could lead to an fd-reuse bug.
>
> Duplicate the outgoing_args fd to avoid this issue.
>
> Suggested-by: Peter Xu <peterx@redhat.com>
> Signed-off-by: Fabiano Rosas <farosas@suse.de>
Reviewed-by: Peter Xu <peterx@redhat.com>
--
Peter Xu
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, (continued)
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Peter Xu, 2024/03/15
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Fabiano Rosas, 2024/03/15
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Peter Xu, 2024/03/15
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Daniel P . Berrangé, 2024/03/19
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Peter Xu, 2024/03/19
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Daniel P . Berrangé, 2024/03/19
- Re: [RFC PATCH v3 3/3] migration: Add fd to FileMigrationArgs, Peter Xu, 2024/03/19
[PATCH v3 1/3] migration/multifd: Ensure we're not given a socket for file migration, Fabiano Rosas, 2024/03/14
[PATCH v3 2/3] migration/multifd: Duplicate the fd for the outgoing_args, Fabiano Rosas, 2024/03/14
- Re: [PATCH v3 2/3] migration/multifd: Duplicate the fd for the outgoing_args,
Peter Xu <=