[Samba] Printing eats my CPU
Gerald Carter
jerry at samba.org
Mon Jul 29 10:24:38 GMT 2002
On Mon, 29 Jul 2002, Franz Sirl wrote:
> At 14:49 29.07.2002, Gerald Carter wrote:
> >On Mon, 29 Jul 2002, Franz Sirl wrote:
> >
> > > Franz.
> > >
> > > * rpc_server/srv_spoolss_nt.c (_spoolss_enumprinterdataex): Don't
> > > try to allocate 0 bytes.
> >
> >Actulayy...try this patch (modified your original one a little
> >to make sure the pointer was NULL if no memory was allocated).
>
> Your patch works equally well here.
>
> >I'm not sure, but I have a gut feeling the marshalling routine
> >may need to be modified to deal with data of length 0. But I
> >really need to see a trace from 2k -> 2k for this. I'll leave it
> >alone for now.
>
> The code in convert_specific_param() handles the length in the same way, so
> I thought it's safe.
I was looking at the code which marshalls the printer data onto the
wire and it looks like there might be some overlapping offsets if tge
data_len == 0. I'll have to experiement some more to be absolutely
certain. But if it is working for you now, then cool :-)
cheers, jerry
---------------------------------------------------------------------
Hewlett-Packard http://www.hp.com
SAMBA Team http://www.samba.org
-- http://www.plainjoe.org
"Sam's Teach Yourself Samba in 24 Hours" 2ed. ISBN 0-672-32269-2
--"I never saved anything for the swim back." Ethan Hawk in Gattaca--
More information about the samba
mailing list