lib/tool/checksum.c fails to compile with -O3

Bug #682745 reported by Diego Biurrun
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HIPL
Confirmed
Low
Unassigned

Bug Description

When configuring and compiling with -O3 in CFLAGS, there are compilation errors.

$ ./configure CFLAGS=-O3
$ make -k
make all-am
make[1]: Entering directory `/home/biurrun/src/hipl/trunk'
  CC lib/tool/checksum.lo
cc1: warnings being treated as errors
lib/tool/checksum.c: In function ‘ipv6_checksum’:
lib/tool/checksum.c:196: error: array subscript is above array bounds
make[1]: *** [lib/tool/checksum.lo] Error 1
  CC hipd/hadb.o
cc1: warnings being treated as errors
In file included from /usr/include/string.h:642,
                 from hipd/hadb.c:65:
In function ‘memset’,
    inlined from ‘lsi_assigned’ at hipd/hadb.c:1388,
    inlined from ‘hip_generate_peer_lsi’ at hipd/hadb.c:1448:
/usr/include/bits/string3.h:86: error: call to __builtin___memset_chk will always overflow destination buffer
make[1]: *** [hipd/hadb.o] Error 1
make[1]: Target `all-am' not remade because of errors.
make[1]: Leaving directory `/home/biurrun/src/hipl/trunk'
make: *** [all] Error 2

Revision history for this message
René Hummen (rene-hummen) wrote :

checksum.c needs to re-written as noted in bug #682323 (https://bugs.launchpad.net/hipl/+bug/682323). This issue should hopefully be resolved by the new code.

Changed in hipl:
status: New → Incomplete
status: Incomplete → Confirmed
importance: Undecided → Low
status: Confirmed → Invalid
status: Invalid → Confirmed
Revision history for this message
Miika Komu (miika-iki) wrote :

I can't repeat this problem. Close?

Miika Komu (miika-iki)
Changed in hipl:
status: Confirmed → Invalid
Revision history for this message
Miika Komu (miika-iki) wrote :

I think this problem affects the OpenWRT port, right? I think I was too hasty to close the bug.

Diego: "To be precise, if you think this is no longer a problem, reenable the
test in tools/hipl_autobuild.sh"

Changed in hipl:
status: Invalid → Confirmed
Revision history for this message
Miika Komu (miika-iki) wrote :

The line in tools/hipl_autobuild.sh is this one:

# Max compile coverage configuration with full optimization
# FIXME: Disabled until the tree compiles with this optimization level.
#compile $FEATURES_ALL CFLAGS="-O3"

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.