diff options
| author | Peter Xu <peterx@redhat.com> | 2024-06-17 15:57:20 -0300 |
|---|---|---|
| committer | Fabiano Rosas <farosas@suse.de> | 2024-06-20 10:42:24 -0300 |
| commit | 1cd93fb0bf8b1fddab4c38e17145cc8776eadaa0 (patch) | |
| tree | f09602edbb8ff11353ffb618095eec870bc6af33 /stubs | |
| parent | 926554c0bfdfbf7b058ed370c2b484e56b126d34 (diff) | |
monitor: Drop monitor_fdset_dup_fd_find/_remove()
Those functions are not needed, one remove function should already
work. Clean it up.
Here the code doesn't really care about whether we need to keep that dupfd
around if close() failed: when that happens something got very wrong,
keeping the dup_fd around the fdsets may not help that situation so far.
Cc: Dr. David Alan Gilbert <dave@treblig.org>
Cc: Markus Armbruster <armbru@redhat.com>
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
[add missing return statement, removal during traversal is not safe]
Signed-off-by: Fabiano Rosas <farosas@suse.de>
Diffstat (limited to 'stubs')
| -rw-r--r-- | stubs/fdset.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/stubs/fdset.c b/stubs/fdset.c index d7c39a28ac..389e368a29 100644 --- a/stubs/fdset.c +++ b/stubs/fdset.c @@ -9,11 +9,6 @@ int monitor_fdset_dup_fd_add(int64_t fdset_id, int flags) return -1; } -int64_t monitor_fdset_dup_fd_find(int dup_fd) -{ - return -1; -} - void monitor_fdset_dup_fd_remove(int dupfd) { } |