Re: Re : [cvsnt] Unknown expansion option ' æ'

Tony Hoyle tmh at nodomain.org
Wed Jun 1 11:06:50 BST 2005


David Somers wrote:
  > The problem is:
> np->data =
> xstrdup((options&&*options)?options+2:(rcs->expand?rcs->expand:"kv"));
> 
> options only contains "o", but I think you are expecting it to contain
> something like "-ko" (because you squirt out options+2... but here options
> only contain "o" so squirting out options+2 results in garbage being written
> instead of valid.
> 
> It might be worth putting an assert like
> assert(options && options[0]=='-' && options[1]=='k')
> before line 5052 as a guard.
> 
> So I guess the problem is upstream in src/import.cpp which is somehow
> setting a bogus options flag.
>
Probably best to do something like:

np->data = 
xstrdup((options&&*options)?((options[0]=='-')?options+2:options):(rcs->expand?rcs->expand:"kv"));

which covers all the bases.

Tony



More information about the cvsnt mailing list