From 110a6f29f0d130753419d5fc5c7b238ab30822ec Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Tue, 2 Feb 2010 16:32:51 -0800 Subject: Fix bug 7063 - Samba 3.4.5 on ubuntu 8.04 64 bit - Core dumps. Reported and found by Martin Hochreiter . Ensure we copy the right amount of registry data into the outgoing buffer. Jeremy. --- source3/rpc_server/srv_spoolss_nt.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'source3') diff --git a/source3/rpc_server/srv_spoolss_nt.c b/source3/rpc_server/srv_spoolss_nt.c index b1513dd329..e2e523d0de 100644 --- a/source3/rpc_server/srv_spoolss_nt.c +++ b/source3/rpc_server/srv_spoolss_nt.c @@ -7634,8 +7634,15 @@ WERROR _spoolss_EnumPrinterData(pipes_struct *p, /* data - counted in bytes */ - if (r->out.data && regval_size(val)) { - memcpy(r->out.data, regval_data_p(val), regval_size(val)); + /* + * See the section "Dynamically Typed Query Parameters" + * in MS-RPRN. + */ + + if (r->out.data && regval_data_p(val) && + regval_size(val) && r->in.data_offered) { + memcpy(r->out.data, regval_data_p(val), + MIN(regval_size(val),r->in.data_offered)); } *r->out.data_needed = regval_size(val); -- cgit