argus client coredump w/ -c flag.

[Carter Bullard]

This patch will solve your reported problem:

dyn-168-208:common carter$ diff ./common/argus_util.c ./common/argus_util.c.new
854c854
<       getoptStr = "a:A:bB:cC:dD:E:e:f:F:g:GhH:iJlLm:M:nN:OpP:qr:R:S:s:t:T:u:U:Vvw:XzZ:";
---
>       getoptStr = "a:A:bB:c:C:dD:E:e:f:F:g:GhH:iJlLm:M:nN:OpP:qr:R:S:s:t:T:u:U:Vvw:XzZ:";

Carter


On Sep 18, 2013, at 2:34 PM, Carter Bullard <carter at qosient.com> wrote:

> Hey Jim,
> Simple fix.  I'll post a patch when I can power up the laptop.
> In argus_util.c, there is a getoptStr that is specified for the various tools.
> The one for "rahisto", needs a colon just after the lowercase 'c', so that getopt() will load up the optarg with a variable.
> 
> Sorry about that!!!
> Carter
> 
> 
> On Sep 18, 2013, at 10:14 AM, "James A. Robinson" <jimr at highwire.stanford.edu> wrote:
> 
>> Looks like the problem is that optarg isn't getting set
>> to a valid location, it appears to be crashing when
>> testing optarg[0] for the value '\\'.
>> 
>> argus-clients-3.0.7.5/common/argus_util.c
>> ...
>>         case 'c': 
>>            if (!(strncmp(parser->ArgusProgramName, "radium", 6))) {
>>               if ((chroot_dir = strdup(optarg)) == NULL)
>>                   ArgusLog (LOG_ERR, "strdup %s", strerror(errno));
>>            } else 
>>            if (!(strncmp(parser->ArgusProgramName, "ragrep", 6))) {
>>               parser->cflag++;
>>            } else {
>>               parser->cflag++;
>>               if (optarg[0] == '\\') {
>> ...
>> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 6837 bytes
Desc: not available
URL: <https://pairlist1.pair.net/pipermail/argus/attachments/20130919/a6c31945/attachment.bin>