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