[patch] make the logging header customisable

Jeremy Allison jra at samba.org
Thu Jun 6 10:21:22 MDT 2013


On Wed, Jun 05, 2013 at 11:23:03PM -0700, Matthieu Patou wrote:

> It's patch made on master, I just pull master and tried to reapply
> and it works

Cool ! That makes things easier :-).

> Well I wanted to avoid dynamic memory allocation and tried to be
> very cautious about the overflow (and I actually did some testing to
> check that no matter we truncate), we could use a static buffer as
> well where the stack is not easily accessible.
> Note that the previous version has also this 200 buffer in the stack.

Yes, I'm not suprised :-). Although maybe we should improve
this.

> Thanks for the feedback, much appreciated that you came back to me !

No problem. I should have some free time to work with you
on this by the end of next week.

Jeremy.


More information about the samba-technical mailing list