[Mauiusers] Possible error in Server.c

[Dave Jackson]

Ake,

  I believe the code is correct.  In previous instances,
ServerProcessArgs() could potentially free a subset of the arguments
based on values found.  Consequently, 'holes' could exist in the array
which is bounded by the termination index found in ArgV.

  We will add comments to this section to clarify.  Thanks for looking
at this!

Dave

On Mon, 2006-01-16 at 11:57 +0100, Åke Sandgren wrote:
> Shouldn't
> =========
>   MSysDoTest();
>  
>   ServerProcessArgs(ArgC,tmpArgV,FALSE);
> 
>   for (aindex = 0;aindex < ArgC;aindex++)
>     {
>     if (ArgV[aindex] == NULL)
>       break;
> 
>     MUFree(&tmpArgV[aindex]);
>     }
>  
> ==========
> in Server.c be
> if (tmpArgV[aindex] == NULL)
> for consistency
> _______________________________________________
> mauiusers mailing list
> mauiusers at supercluster.org
> http://www.supercluster.org/mailman/listinfo/mauiusers