Elliot,

I'm just scanning through your patches. I will apply them to the
development branch (4.7 stuff) - but I'm doing them one by one, since you
are highlighting good problems for us to fix in a decent way if we can
(and if they are important).

Thanks very much!

- Peter -

On Thu, 3 Sep 1998, Elliot Lee wrote:

> I had sent this earlier, but it must have gotten lost, because I
> just downloaded 4.6.5, and this patch applies cleanly to it, and I hadn't
> seen any reply saying why the patch was a Bad Idea.
> 
> Here's the original message text:
> ---------------------------------
> Here's a patch to fix all the problems that an various tools
> found at compile time in coda 4.6.0...
> 
> Notes:
>         - There are some sprintfs that had an extra argument that
>           was supposed to be displayed in an error message. Rather
>           than make the format string include the argument, I just
>           removed the argument altogether (to avoid buffer overflow
>           exploit possibilities). You really need to define a standard
>           way of using snprintf() in coda, though.
> 
>         - In places where I didn't know how to fix things, I tried
>           to put comments (i.e. /* XXX incompatible types */) to
>           make it easier for Someone Else to fix the problem.
> 
>         - I know that one of the "return -1"s broke venus;
>           however it is a fairly easy problem to spot & fix, and it is
>           better than depending on the random return value to be zero.
> 
> As soon the right return values are put in so venus starts up, I will try
> to get someone to run it through purify or insure++ and find more bugs :-)
> 
> Hope this helps,
> -- Elliot
> Progress (n.): The process through which Usenet has evolved from smart
> people in front of dumb terminals to dumb people in front of smart
> terminals.  -- obs_at_burnout.demon.co.uk
>