This file generates a warning because the left side of a variable
assignment is commented out. Keep the effect of the line
(incrementing packet) but remove the unused casting and
reference, and remove the comments that date from the original
IBM source.
Leave a new comment in place in case the information is useful.
Adjust the Makefile and README.WARNINGS to account for the change.
Change-Id: I944e1c488e39411d32d700cba2d3ef567eddddb4
Reviewed-on: http://gerrit.openafs.org/804
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
butc/dump.c : all : pointer types (XXX)
comerr/error_table.c : all : Autogenerated file with unused labels
kauth/kaserver.c : all : ExtendedCellInfo
-kauth/krb_udp.c : all : "Value computed is not used" - wierd
kauth/admin_tools.c : all : ubik_Call nonsense
kauth/authclient.c : strict-proto : ubik_Call nonsense
: all : ubik_Call nonsense
kdb.o: kdb.c AFS_component_version_number.o
krb_udp.o: krb_udp.c kaserver.h kautils.h kauth.h prot.h ${TOP_INCDIR}/lwp.h AFS_component_version_number.o
- $(CC) $(CFLAGS) @CFLAGS_NOERROR@ -c $<
+ $(CC) $(CFLAGS) -c $<
krb_udp: krb_udp.o libkauth.a $(KLIBS)
${CC} ${LDFLAGS} -o krb_udp krb_udp.o libkauth.a $(KLIBS)
char *packet;
des_key_schedule schedule;
afs_int32 cksum;
- /* unsigned char time_msec; */
afs_int32 time_sec;
int byteOrder = pkt->byteOrder;
return KABADTICKET;
packet += strlen(packet) + 1;
getint(cksum);
- /* time_msec = */ *(unsigned char *)packet++;
+ /* Comments in the original IBM source suggest this byte was/is "time_msec" */
+ packet++;
getint(time_sec);
if ((packet - auth) > authLen)
return KABADTICKET;