Yup. usage() is calling itself with no termination condition - and what's more, it is tail recursing. It should be

        void
        usage(void)
        {
fprint(2, "usage: sshserve [-A authlist] [-c cipherlist] client-ip- address\n");
                exits("usage");
        }

to keep with the Plan 9 program. Enjoy!

When this QEMU-on-Leopard-freezes-randomly bug gets fixed, I can fix it and any other dangers.

On Apr 15, 2008, at 6:05 PM, Kenji Arisawa wrote:

Hello,

In sshserve.c:

void
usage(void)
{
fprint(2, "usage: sshserve [-A authlist] [-c cipherlist] client-ip- address\n");
        usage();
}

This code can make endless loop. Right?

Kenji Arisawa




Reply via email to