diff options
author | Jeremy Allison <jra@samba.org> | 2007-02-14 17:17:32 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:17:57 -0500 |
commit | 6663fa6bdcd7098444d76b9f8413d9088588244b (patch) | |
tree | eeef43e7845b34e037a00bd38057af736e53fcf5 | |
parent | 2d675293d9356099b334b9d1159fcbf4e76624ac (diff) | |
download | samba-6663fa6bdcd7098444d76b9f8413d9088588244b.tar.gz samba-6663fa6bdcd7098444d76b9f8413d9088588244b.tar.bz2 samba-6663fa6bdcd7098444d76b9f8413d9088588244b.zip |
r21346: FIXME ! Our parsing here is wrong I think,
but for a level3 it makes no sense for
ptr_sec_desc to be NULL. JRA. Based on
a Vista sniff from Martin Zielinski <mz@seh.de>.
Jerry - part of the Vista patchset.
Jeremy.
(This used to be commit 60e26a9039fbe0fd632e306bf545331195fb1ce6)
-rw-r--r-- | source3/rpc_parse/parse_spoolss.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/source3/rpc_parse/parse_spoolss.c b/source3/rpc_parse/parse_spoolss.c index eeadcd1e29..44d35ec24b 100644 --- a/source3/rpc_parse/parse_spoolss.c +++ b/source3/rpc_parse/parse_spoolss.c @@ -3913,7 +3913,16 @@ BOOL spoolss_io_q_setprinter(const char *desc, SPOOL_Q_SETPRINTER *q_u, prs_stru } case 3: { - ptr_sec_desc = q_u->info.info_3->secdesc_ptr; + /* FIXME ! Our parsing here is wrong I think, + * but for a level3 it makes no sense for + * ptr_sec_desc to be NULL. JRA. Based on + * a Vista sniff from Martin Zielinski <mz@seh.de>. + */ + if (UNMARSHALLING(ps)) { + ptr_sec_desc = 1; + } else { + ptr_sec_desc = q_u->info.info_3->secdesc_ptr; + } break; } } |