Latest Time Machine Patch

Ralph Böhme slow at samba.org
Wed Oct 4 08:01:19 UTC 2017


On Tue, Oct 03, 2017 at 10:58:40PM +0000, Jeremy Allison wrote:
> On Tue, Oct 03, 2017 at 06:57:35PM -0400, Kevin Anderson wrote:
> > > Yes, these look good to me. Just one request - did you check this
> > > under valgrind to ensure it's clean ? If so then RB+ from me.
> > >
> > 
> > I've never run valgrind before so this was the first time attempting to do so.
> > 
> > I ran valgrind --leak-check=yes <smbd> both with the Avahi patch and
> > without it (including starting a backup). The output for "definitely
> > lost" remained the same at 2 bytes in 1 block. Should I compare other
> > numbers as well?
> > Kevin Anderson
> 
> Nope, sounds good - thanks for checking ! Can I get a second
> Team reivewer ?

pushed. And sorry for the sloppy review and not insisting on the memory
allocation checks...

Cheerio!
-slow



More information about the samba-technical mailing list