This is an automated email from the git hooks/post-receive script.

guillem pushed a commit to branch master
in repository dpkg.

View the commit online:
https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=9b8f4296cf0b0f3a4aecc59d5d96d2b5f3ad2218

commit 9b8f4296cf0b0f3a4aecc59d5d96d2b5f3ad2218
Author: Guillem Jover <guil...@debian.org>
AuthorDate: Tue Aug 25 01:13:12 2020 +0200

    u-a: Refactor alternative database context freeing into a new function
    
    Changelog: internal
---
 utils/update-alternatives.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/utils/update-alternatives.c b/utils/update-alternatives.c
index 71e4879d6..b98fbe1a6 100644
--- a/utils/update-alternatives.c
+++ b/utils/update-alternatives.c
@@ -1229,6 +1229,14 @@ struct altdb_context {
        jmp_buf on_error;
 };
 
+static void
+altdb_context_free(struct altdb_context *ctx)
+{
+       if (ctx->fh)
+               fclose(ctx->fh);
+       free(ctx->filename);
+}
+
 static int
 altdb_filter_namelist(const struct dirent *entry)
 {
@@ -1464,9 +1472,7 @@ alternative_load(struct alternative *a, enum altdb_flags 
flags)
        }
 
        if (setjmp(ctx.on_error)) {
-               if (ctx.fh)
-                       fclose(ctx.fh);
-               free(ctx.filename);
+               altdb_context_free(&ctx);
                alternative_reset(a);
                return false;
        }

-- 
Dpkg.Org's dpkg

Reply via email to