On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen
| Did you test this change? Verily I'm not sure it's a proper bug, but I | kept it in my list so that someone (bouyer@?) could investigate. This is why I committed the fix in two pieces. The first one is obviously right. The second one is probably correct too, and if you notice there is also another if statement below that does not free either, but returns EINVAL. Didn't your code checker flag that? The only change I did not apply was a false positive (jumping inside a loop!?!?) which was disgusting code. christos
