summaryrefslogtreecommitdiffstats
path: root/source/printing
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2002-12-04 21:28:23 +0000
committerJeremy Allison <jra@samba.org>2002-12-04 21:28:23 +0000
commita2ab78d68d269e77d55bf4b6929cf512a28f0708 (patch)
tree573ff6175dcc41b6feab07779b9aba5d831d73b0 /source/printing
parent974822526f90aee9b43e75fc7fd5d48fe91add99 (diff)
downloadsamba-a2ab78d68d269e77d55bf4b6929cf512a28f0708.tar.gz
samba-a2ab78d68d269e77d55bf4b6929cf512a28f0708.tar.xz
samba-a2ab78d68d269e77d55bf4b6929cf512a28f0708.zip
Janitor for tpot yet again... Merging in memory leak fix.
Jeremy.
Diffstat (limited to 'source/printing')
-rw-r--r--source/printing/nt_printing.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source/printing/nt_printing.c b/source/printing/nt_printing.c
index e37238c2ce9..00d1eaacd3f 100644
--- a/source/printing/nt_printing.c
+++ b/source/printing/nt_printing.c
@@ -2830,6 +2830,7 @@ static int unpack_values(NT_PRINTER_DATA *printer_data, char *buf, int buflen)
regval_ctr_addvalue( &printer_data->keys[key_index].values, valuename, type, data_p, size );
+ SAFE_FREE(data_p); /* 'B' option to tdb_unpack does a malloc() */
DEBUG(8,("specific: [%s:%s], len: %d\n", keyname, valuename, size));
}
@@ -4382,9 +4383,8 @@ BOOL nt_printing_getsec(TALLOC_CTX *ctx, const char *printername, SEC_DESC_BUF *
fstring key;
char *temp;
- if ((temp = strchr(printername + 2, '\\'))) {
+ if (strlen(printername) > 2 && (temp = strchr(printername + 2, '\\')))
printername = temp + 1;
- }
/* Fetch security descriptor from tdb */