Re: cvs admin segfaults

[christos%astron.com@localhost (Christos Zoulas)]

In article <4A0C5F83.4050607%marples.name@localhost>,
Roy Marples  <roy%marples.name@localhost> wrote:
>-=-=-=-=-=-
>
>cvs admin -m1.58:"`cat /tmp/m`" network
>segfault
>bt
>
>#0  0x000000000040731e in admin (argc=4, argv=0x7f7ffc31a080)
>     at /usr/src/external/gpl2/xcvs/dist/src/admin.c:330
>#1  0x000000000042dd11 in main (argc=4, argv=0x7f7ffc31a080)
>     at /usr/src/external/gpl2/xcvs/dist/src/main.c:1167
>
>Seems that the config object isn't allocated at this point, the attached 
>patch fixes this issue. I can't easily see where config is loaded, so I 
>don't know if it's correct.
>

That is strange, config should be allocated at that point, but more seriously
this bypasses the check...

christos