smbd panic at find_oplock_types().

Jeremy Allison jra at samba.org
Tue Sep 9 18:02:44 MDT 2014


On Tue, Sep 09, 2014 at 01:12:35PM +0200, Volker Lendecke wrote:
> 
> Remember that INTERNAL_OPEN_ONLY has the effect that this request will
> never be deferred. For me that's the main use of this flag: Not all
> callers are prepared to handle open_was_deferred. We need an indication
> to open_file_ntcreate to never defer a call. Or even better, we need to
> replace the existing deferral mechanism with a tevent_req based design,
> but that's certainly a future project.

Yes, not deferring INTERNAL_OPEN_ONLY requests is probably
the more important effect. However, I still think that
removing that flag externally to open.c and using req==NULL
to mean the same thing makes it clearer to callers (they
don't have to set two paramters that mean the same
thing).

> What about the attached patch? This fixes batch26. I haven't
> run an autobuild yet, starting that in a minute.

Let me think about it some more and take a close
look.

On initial evalutation it looks like the right fix,
but you know how tricky this stuff is :-).

We need to commit batch26 along with this fix
anyway - can you give Hemmanth's batch26
code a 'Reviewed-by' ?

Cheers,

	Jeremy.


More information about the samba-technical mailing list