smbd: Avoid double-free in get_print_db_byname
authorVolker Lendecke <vl@samba.org>
Tue, 8 Jul 2014 12:30:54 +0000 (14:30 +0200)
committerDavid Disseldorp <ddiss@samba.org>
Tue, 8 Jul 2014 20:27:03 +0000 (22:27 +0200)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Autobuild-User(master): David Disseldorp <ddiss@samba.org>
Autobuild-Date(master): Tue Jul  8 22:27:03 CEST 2014 on sn-devel-104

source3/printing/printing_db.c

index ecb8ff6463cffa579f2e86bd02f4b7c17ad154c2..b721317e5d6816a14b8b54dda9381e5feae4f1ac 100644 (file)
@@ -65,9 +65,9 @@ struct tdb_print_db *get_print_db_byname(const char *printername)
                        if (p->ref_count)
                                continue;
                        if (p->tdb) {
-                               if (tdb_close(print_db_head->tdb)) {
+                               if (tdb_close(p->tdb)) {
                                        DEBUG(0,("get_print_db: Failed to close tdb for printer %s\n",
-                                                               print_db_head->printer_name ));
+                                                               p->printer_name ));
                                        return NULL;
                                }
                        }