[PATCHSET] Fix some memory leaks

Andreas Schneider asn at samba.org
Thu Jun 23 06:25:57 UTC 2016


On Wednesday, 22 June 2016 20:24:30 CEST Uri Simchoni wrote:
> On 06/22/2016 08:11 PM, Alexander Bokovoy wrote:
> > On Wed, 22 Jun 2016, Uri Simchoni wrote:
> >> Works with custom-built master and setting of PYTHONPATH.
> > 
> > I don't think you should be setting PYTHONPATH if the package is
> > installed. If xattr_native.so is placed in the right part of the Python
> > tree, there shouldn't be any need for additional PYTHONPATH.
> > 
> > $ python -c "import sys; print '/usr/lib64/python2.7/site-packages' in
> > sys.path" True
> 
> What I actually meant to say by this was that it's probably a packaging
> issue, not a Samba issue - that importing the manually-built module
> works whereas importing the packaged module (without the PYTHONPATH)
> doesn't.

The module is not built and thus not packaged. Günther has a patch for this 
which is only in master. So this patch needs to be backported to 4.4 and 4.3.

Something totall unrelated: Could someone also review the patch this mail 
thread is about :)



-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list