>Number: 5268
>Category: mod_jserv
>Synopsis: jserv_mmap.c fails to compile
>Confidential: no
>Severity: non-critical
>Priority: medium
>Responsible: jserv
>State: open
>Class: sw-bug
>Submitter-Id: apache
>Arrival-Date: Sun Nov 7 16:20:01 PST 1999
>Last-Modified:
>Originator: [EMAIL PROTECTED]
>Organization:
apache
>Release: Apache/1.3.9 (Unix) ApacheJServ/1.1b3
>Environment:
SunOS 5.7 Generic_106541-08 sun4u sparc SUNW,Ultra-2
java version "1.1.7"
cc: WorkShop Compilers 4.2 30 Oct 1996 C 4.2
>Description:
The following two procedures are not supposed to return a value but they do:
void jserv_setalive(jserv_config *cfg, jserv_host *cur) {
/* we do not override admin commands (state=SHUTDOWN*) */
return jserv_changestate(cfg, cur, "-", '+');
}
void jserv_setdead(jserv_config *cfg, jserv_host *cur) {
/* we do not override admin commands (state=SHUTDOWN*) */
return jserv_changestate(cfg, cur, "+", '-');
}
-----------------------------------------
In procedure jserv_getnext_host the following cast creates a problem since
it's not an lvalue:
(int)shmhost->opaque += 1;
>How-To-Repeat:
N/A
>Fix:
Remove the "return"s and the (int) cast.
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, you need]
[to include <[EMAIL PROTECTED]> in the Cc line and make sure the]
[subject line starts with the report component and number, with ]
[or without any 'Re:' prefixes (such as "general/1098:" or ]
["Re: general/1098:"). If the subject doesn't match this ]
[pattern, your message will be misfiled and ignored. The ]
["apbugs" address is not added to the Cc line of messages from ]
[the database automatically because of the potential for mail ]
[loops. If you do not include this Cc, your reply may be ig- ]
[nored unless you are responding to an explicit request from a ]
[developer. Reply only with text; DO NOT SEND ATTACHMENTS! ]
