Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Make warnings in Debian 7.0 #6058

Closed
shakaran opened this issue Aug 15, 2013 · 2 comments
Closed

Make warnings in Debian 7.0 #6058

shakaran opened this issue Aug 15, 2013 · 2 comments

Comments

@shakaran
Copy link

I am getting this warnings (openssl mostly) running make in Debian 7.0. It compiles node, but just putting here the errors if helps to prevent some future derivate errors:

../deps/cares/src/ares_process.c: In function 'read_udp_packets':
../deps/cares/src/ares_process.c:472:37: warning: pointer targets in passing argument 6 of 'recvfrom' differ in signedness [-Wpointer-sign]
In file included from ../deps/cares/src/setup_once.h:95:0,
                 from ../deps/cares/src/ares_setup.h:218,
                 from ../deps/cares/src/ares_process.c:18:
/usr/include/x86_64-linux-gnu/sys/socket.h:166:16: note: expected 'socklen_t * __restrict__' but argument is of type 'ares_socklen_t *'

../deps/openssl/openssl/crypto/modes/gcm128.c: In function 'CRYPTO_gcm128_setiv':
../deps/openssl/openssl/crypto/modes/gcm128.c:813:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:821:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c: In function 'CRYPTO_gcm128_encrypt':
../deps/openssl/openssl/crypto/modes/gcm128.c:916:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:947:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:952:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:967:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:972:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1001:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1019:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c: In function 'CRYPTO_gcm128_decrypt':
../deps/openssl/openssl/crypto/modes/gcm128.c:1063:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1097:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1102:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1115:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1120:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1149:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1170:5: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c: In function 'CRYPTO_gcm128_encrypt_ctr32':
../deps/openssl/openssl/crypto/modes/gcm128.c:1215:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1237:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1252:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1272:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c: In function 'CRYPTO_gcm128_decrypt_ctr32':
../deps/openssl/openssl/crypto/modes/gcm128.c:1314:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1339:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1365:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
../deps/openssl/openssl/crypto/modes/gcm128.c:1376:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
@bnoordhuis
Copy link
Member

Yeah, I know about them. They're from our bundled dependencies, we don't have much influence on that apart from disabling the warnings.

Regarding the type-punning warnings, they may be false positives. You're probably compiling with an older version of gcc (Debian 7 - is that 4.4?). There's a pretty good chance that the warnings go away when you upgrade, gcc's type punning heuristics have improved a great deal.

@shakaran
Copy link
Author

Where should I report for the bundle dependencies warnings?

I am using:

gcc --version
gcc (Debian 4.7.2-5) 4.7.2

And OpenSSL 1.0.1e 11 Feb 2013

I think that the software is pretty updated, maybe the bundle dependencies are not updated. Could configure show a warning if the openssl version is minor or some bundle will show this warnings?

gibfahn pushed a commit to ibmruntimes/node that referenced this issue Apr 20, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
gibfahn pushed a commit to ibmruntimes/node that referenced this issue Apr 27, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
utterstep pushed a commit to lutik-inc/node that referenced this issue Jun 1, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
richardlau pushed a commit to ibmruntimes/node that referenced this issue Jun 17, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
richardlau pushed a commit to ibmruntimes/node that referenced this issue Jun 17, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
richardlau pushed a commit to ibmruntimes/node that referenced this issue Jun 29, 2016
* Modify tools/license-builder.sh to support ICU 57.1's plain text
license. (Separate issue to add ICU 57.1 in nodejs#6058)
* Update/regenerate LICENSE to include ICU 57.1's license
* Note that because the tool was rerun, the change in nodejs#6065 is already
included here.

PR-URL: nodejs/node#6068
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants