[IRCServices Coding] svcs5

[Andrew Church]

>this may try to free something that is not malloced (if i get this right)
>when it is called from makechan() (free(NULL) works under glibc, but i do
>n't
>know how portable/standard it is)

     I'm pretty sure that's standard, and if it turns out not to be I'll
just write my own free() because the rest of the code assumes it works
that way.

>finaly in chanserv/access-levels.c, in do_levels():
[snip]
>the marked line is where i get the coredump...
>it doesn't make any sense.. 
>if ci->levels is NULL, then reset_levels is not called...
>so it remains NULL and then ci->levels[] is used

     The test on ci->levels was backwards in sense.  Fixed, thanks.

  --Andrew Church
    achurch at achurch.org
    http://achurch.org/