Great Circle Associates Majordomo-Users
(February 1995)
 

Indexed By Date: [Previous] [Next] Indexed By Thread: [Previous] [Next]

Subject: Re: Compilation errors in wrapper (majordomo 1.93)
From: "Franklin R. Jones" <frjones @ nma . mnet . uswest . com>
Date: Mon, 27 Feb 1995 13:18:29 -0700
To: Frostie @ wyoming . com (Frostie Sprout)
Cc: majordomo-users @ greatcircle . com
In-reply-to: Your message of "Mon, 27 Feb 1995 11:47:50 MST." <199502271847.LAA15290@calamity.wyoming.com>

Frostie,
> I'm compiling wrapper in majordomo 1.93 on a Linux box. I've changed the
> line in wrapper.c
> 
> char setgroups_used = "setgroups_was_included"; /* give strings a hint */
> 
> to
> 
> char *setgroups_used = "setgroups_was_included"; /* give strings a hint */
> 
> 
> but I come up with another error during compilation:
> wrapper.c:65: warning: comparison between pointer and integer
> 
> Line 65 of wrapper.c:
> 
> 65:    if (STRCHR(argv[1], '/') != (char *) NULL) {
> 66:        /* this error message is intentionally cryptic */
> 67:        fprintf(stderr, "%s: error: insecure usage\n", argv[0]);
> 68:        exit(2);
> 
> I've been using wrapper with the errors, no problems that I'm aware of...
> 
> How do I fix this (or do I need to)?
> Frostie Sprout

I had the same errors under Solaris 5.4. I changed 

> 65:    if (STRCHR(argv[1], '/') != (char *) NULL) {

to:

=> 65:    if (STRCHR(argv[1], '/') != NULL) {

and it got rid of the error. depends on what your compiler did with
the mapping as to whether it will cause problems, most likely not as
it produced a warning.

fj..





References:
Indexed By Date Previous: Re: MajorDomo & HTML Interface?
From: ks@acn.purdue.edu (Kirk D. Smith)
Next: Re: Compilation errors in wrapper (majordomo 1.93)
From: Bryan Curnutt <curnutt@Stoner.COM>
Indexed By Thread Previous: Compilation errors in wrapper (majordomo 1.93)
From: Frostie@wyoming.com (Frostie Sprout)
Next: Re: Compilation errors in wrapper (majordomo 1.93)
From: Bryan Curnutt <curnutt@Stoner.COM>

Google
 
Search Internet Search www.greatcircle.com