External Sync VFS module

Ira Cooper ira at wakeful.net
Thu Apr 28 00:46:34 UTC 2016


Please, can we allocate a small amount of memory and not have this connect
and disconnect constantly... It just looks ugly.

Also do we need a license at the top of the python daemon?

Is the fact that this daemon is single threaded going to cause a problem?

Just some thoughts looking at it,

-Ira

On Wed, Apr 27, 2016 at 8:04 PM, Jeremy Allison <jra at samba.org> wrote:

> On Tue, Apr 26, 2016 at 04:51:51PM -0500, Alex Austin wrote:
> > Hello,
> > I've written a Samba VFS module to run an external file-sync program, and
> > would appreciate feedback.
>
> Currently you call the SMB_VFS_NEXT_XXX() function,
> but then always call the sendsyncmsg() without
> regard to whether the VFS call succeeded or failed.
>
> Don't you need to check the return first, and only
> call sendsyncmsg() on success ?
>
> Jeremy.
>
>


More information about the samba-technical mailing list