[PATCH 1/2] kernel/tests: Implement safe local strcat function.

Saulius Krasuckas saulius2 at ar.fi.lt
Tue Jul 4 14:07:03 CDT 2006


* On Tue, 4 Jul 2006, Dmitry Timoshkov wrote:
> * "Saulius Krasuckas" wrote:
> > 
> > It was one "if" that I removed in my patch.  Maybe you mean this way?
> 
> Yes, that's a perfect way of checking for a possible buffer overflow, 
> there is no need to complicate things.

It may be perfect from some POVs, but it creates some conditional 
branching which I was thinking we're are trying to avoid in WRT suite, 
for example branching used in checking for version-dependent stuff.

> strcat wrapper creates an illusion that it solves some problem when it 
> actually doesn't.

So you agree there is a problem which isn't solved in current test code?



More information about the wine-devel mailing list