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=2be0e88b61c5a6d2d15860eb68100248c105eab3

The following commit(s) were added to refs/heads/master by this push:
     new 2be0e88b6 libdpkg: Reset the rootnode to NULL after free()ing it
2be0e88b6 is described below

commit 2be0e88b61c5a6d2d15860eb68100248c105eab3 (HEAD -> master)
Author: Guillem Jover <[email protected]>
AuthorDate: Tue Jan 5 13:50:03 2021 +0100

    libdpkg: Reset the rootnode to NULL after free()ing it
    
    We are testing whether rootnode is NULL, in part to placate code
    analyzers not understanding the structure of the code, and to make
    it more robust, so after free()ing it we should reset it to NULL.
    
    Fixes: commit b7556b76e81c4b02d14dd030cf128503f237f922
    Warned-by: coverity
    Changelog: silent
---
 lib/dpkg/treewalk.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/dpkg/treewalk.c b/lib/dpkg/treewalk.c
index f66823e49..00ce66cc5 100644
--- a/lib/dpkg/treewalk.c
+++ b/lib/dpkg/treewalk.c
@@ -495,6 +495,7 @@ treewalk_next(struct treeroot *tree)
 
                if (tree->curdir == NULL) {
                        treenode_free_node(tree->rootnode);
+                       tree->rootnode = NULL;
                        break;
                }
        }

-- 
Dpkg.Org's dpkg

Reply via email to