From 262b5f1f532a8bd39bd0810d4c388e3fcf6051dd Mon Sep 17 00:00:00 2001 From: Andreas Persson Date: Sat, 29 Mar 2025 08:37:01 +0100 Subject: [PATCH] envy24control: fix file descriptor leaks in profiles File descriptors were leaked when "Save active profile" was pressed. Add the missing calls to close. Signed-off-by: Andreas Persson --- envy24control/profiles.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/envy24control/profiles.c b/envy24control/profiles.c index 17a418c..7853628 100644 --- a/envy24control/profiles.c +++ b/envy24control/profiles.c @@ -1144,13 +1144,16 @@ int save_restore(const char * const operation, const int profile_number, const i fprintf(stderr, "Cannot save settings for card '%d' in profile '%d'.\n", card_number, profile_number); return -errno; } + close(res); unlink(cfgfile); } else { + close(res); if ((res = open(cfgfile, O_RDWR | 0400000 /* O_NOFOLLOW */, FILE_CREA_MODE)) < 0) { fprintf(stderr, "Cannot open configuration file '%s' for writing.\n", cfgfile); fprintf(stderr, "Cannot save settings for card '%d' in profile '%d'.\n", card_number, profile_number); return -errno; } + close(res); } res = save_profile(profile_number, card_number, profile_name, cfgfile); } else if (!strcmp(operation, ALSACTL_OP_RESTORE)) {