summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGünther Deschner <gd@samba.org>2010-04-07 23:34:36 +0200
committerGünther Deschner <gd@samba.org>2010-04-07 23:53:18 +0200
commit34ada1d540b0eedbe1dd573a47146f0309ed4875 (patch)
tree1be770c3c442e14d6b6af6859a06a314e175e9a9
parent7f3116a63d7d91f4c0d26adf8fcdef0a5a957971 (diff)
downloadsamba-34ada1d540b0eedbe1dd573a47146f0309ed4875.tar.gz
samba-34ada1d540b0eedbe1dd573a47146f0309ed4875.tar.bz2
samba-34ada1d540b0eedbe1dd573a47146f0309ed4875.zip
s3-spoolss: Fix potential memleak in _spoolss_AddForm and _spoolss_SetForm.
Guenther
-rw-r--r--source3/rpc_server/srv_spoolss_nt.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/rpc_server/srv_spoolss_nt.c b/source3/rpc_server/srv_spoolss_nt.c
index bb14c597da..c5c2dfbf1b 100644
--- a/source3/rpc_server/srv_spoolss_nt.c
+++ b/source3/rpc_server/srv_spoolss_nt.c
@@ -7915,7 +7915,8 @@ WERROR _spoolss_AddForm(pipes_struct *p,
p->server_info->ptok,
lp_printer_admin(snum))) {
DEBUG(2,("_spoolss_Addform: denied by insufficient permissions.\n"));
- return WERR_ACCESS_DENIED;
+ status = WERR_ACCESS_DENIED;
+ goto done;
}
/* can't add if builtin */
@@ -8079,7 +8080,8 @@ WERROR _spoolss_SetForm(pipes_struct *p,
p->server_info->ptok,
lp_printer_admin(snum))) {
DEBUG(2,("_spoolss_Setform: denied by insufficient permissions.\n"));
- return WERR_ACCESS_DENIED;
+ status = WERR_ACCESS_DENIED;
+ goto done;
}
/* can't set if builtin */