FW: Delivery Status Notification (Failure)

[Volker Lendecke]

On Wed, Aug 21, 2013 at 11:20:08PM -0700, Matthieu Patou wrote:
> On 08/21/2013 11:05 PM, Volker Lendecke wrote:
> >On Thu, Aug 22, 2013 at 06:44:00AM +0200, Volker Lendecke wrote:
> >>On Thu, Aug 22, 2013 at 04:37:48AM +0000, luke morrison wrote:
> >>>Hello Team,
> >>>This is Luke in response to Matthieu's debugging session
> >>>about the weird Samba error when trying to make a GPO "the
> >>>command was an incorrect length". Here are the associated
> >>>logs, at higher log levels. It was a reoccuring issue that
> >>>Matthieu helped fix, and it happened only when I tried to
> >>>make a GPO.
> >>Send them to me directly. And compress them first
> >>please.
> >The only way I can see this happen is with the source4 based
> >ntvfs file server. Is it possible that you use that instead
> >of the source3 based smbd?
> >
> >The offending lines might be in
> >source4/smb_server/smb2/fileinfo.c:
> >
> >         if (op->info->in.output_buffer_length < op->info->out.blob.length) {
> >                 smb2srv_send_error(req, NT_STATUS_INFO_LENGTH_MISMATCH);
> >                 return;
> >         }
> >
> >Which file server are you using?
> I think I checked that there was some smbd process running and that
> the services line included the s3fs, but to be 100% it's better to
> get the smb.conf.
> 
> What about
> 507         if (state->out_output_buffer.length >
> in_output_buffer_length) {
> 508                 tevent_req_nterror(req,
> NT_STATUS_INFO_LENGTH_MISMATCH);
> 509                 return tevent_req_post(req, ev);
> 510         }
> in source3/smbd/smb2_getinfo.c ?

Ok, right. I was confused by the fact that in the
smbd_do_query_security_desc branch we already set the
status. This is definitely a bug.

Sorry for not seeing it.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de