On Nov 20, 2011, at 6:10 PM, Christos Zoulas wrote:
> On Nov 20, 10:02am, hann...@eis.cs.tu-bs.de ("J. Hannken-Illjes") wrote:
> -- Subject: Re: CVS commit: src/sys/kern
>
> |
> | On Nov 19, 2011, at 5:11 PM, Christos Zoulas wrote:
> |
> | > Module Name: src
> | > Committed By: christos
> | > Date: Sat Nov 19 16:11:24 UTC 2011
> | >
> | > Modified Files:
> | > src/sys/kern: cnmagic.c
> | >
> | > Log Message:
> | > PR/45633: Christian Biere: Don't access byte after NUL when setting magic.
> |
> | This cannot work:
> |
> | if (c == '\0')
> | return EINVAL;
> | switch (c) {
> | case '\0':
> | ...
> | return 0; <== This statement looks unreachable.
> |
> | All attempts to set cnmagic result in EINVAL.
>
> The patch was amended. Do you still have an issue?
No, Rev. 1.13 works fine -- sorry for the noise ...
--
Juergen Hannken-Illjes - hann...@eis.cs.tu-bs.de - TU Braunschweig (Germany)
