diff options
author | davidben <davidben@chromium.org> | 2014-11-05 12:11:41 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-11-05 20:13:06 +0000 |
commit | d6461f32e4e9c7274083604cd30290c1739b91fb (patch) | |
tree | d9996a45d98a4a8771c7533d434992a4e6fa9717 /third_party/boringssl | |
parent | 88a7a0f353388c52621aad760337aa0a8c0e0084 (diff) | |
download | chromium_src-d6461f32e4e9c7274083604cd30290c1739b91fb.zip chromium_src-d6461f32e4e9c7274083604cd30290c1739b91fb.tar.gz chromium_src-d6461f32e4e9c7274083604cd30290c1739b91fb.tar.bz2 |
Disable size_t truncation warning (C4267) on BoringSSL Win64.
There's a ton of them; OpenSSL is very far from being size_t-clean. Disable
the warning (as in the standalone build's cmake setup) until we get a handle
on it in BoringSSL.
BUG=429039, 167187
Review URL: https://codereview.chromium.org/684093011
Cr-Commit-Position: refs/heads/master@{#302857}
Diffstat (limited to 'third_party/boringssl')
-rw-r--r-- | third_party/boringssl/BUILD.gn | 6 | ||||
-rw-r--r-- | third_party/boringssl/boringssl.gyp | 3 | ||||
-rw-r--r-- | third_party/boringssl/boringssl_tests.gypi | 66 | ||||
-rw-r--r-- | third_party/boringssl/update_gypi_and_asm.py | 3 |
4 files changed, 78 insertions, 0 deletions
diff --git a/third_party/boringssl/BUILD.gn b/third_party/boringssl/BUILD.gn index 5dffa65..0ee12f9 100644 --- a/third_party/boringssl/BUILD.gn +++ b/third_party/boringssl/BUILD.gn @@ -61,6 +61,12 @@ component("boringssl") { "src/crypto", ] + if (is_win) { + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + cflags += [ "/wd4267"] + } + if (cpu_arch == "x64") { if (is_mac) { sources += gypi_values.boringssl_mac_x86_64_sources diff --git a/third_party/boringssl/boringssl.gyp b/third_party/boringssl/boringssl.gyp index c3760c0..238cb99 100644 --- a/third_party/boringssl/boringssl.gyp +++ b/third_party/boringssl/boringssl.gyp @@ -17,6 +17,9 @@ 'BORINGSSL_IMPLEMENTATION', 'BORINGSSL_NO_STATIC_INITIALIZER', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], 'conditions': [ ['component == "shared_library"', { 'defines': [ diff --git a/third_party/boringssl/boringssl_tests.gypi b/third_party/boringssl/boringssl_tests.gypi index 04f0c43..fe7eb61 100644 --- a/third_party/boringssl/boringssl_tests.gypi +++ b/third_party/boringssl/boringssl_tests.gypi @@ -15,6 +15,9 @@ 'sources': [ 'src/crypto/base64/base64_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_bio_test', @@ -25,6 +28,9 @@ 'sources': [ 'src/crypto/bio/bio_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_bn_test', @@ -35,6 +41,9 @@ 'sources': [ 'src/crypto/bn/bn_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_bytestring_test', @@ -45,6 +54,9 @@ 'sources': [ 'src/crypto/bytestring/bytestring_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_aead_test', @@ -55,6 +67,9 @@ 'sources': [ 'src/crypto/cipher/aead_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_cipher_test', @@ -65,6 +80,9 @@ 'sources': [ 'src/crypto/cipher/cipher_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_dh_test', @@ -75,6 +93,9 @@ 'sources': [ 'src/crypto/dh/dh_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_dsa_test', @@ -85,6 +106,9 @@ 'sources': [ 'src/crypto/dsa/dsa_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_example_mul', @@ -95,6 +119,9 @@ 'sources': [ 'src/crypto/ec/example_mul.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_ecdsa_test', @@ -105,6 +132,9 @@ 'sources': [ 'src/crypto/ecdsa/ecdsa_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_err_test', @@ -115,6 +145,9 @@ 'sources': [ 'src/crypto/err/err_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_example_sign', @@ -125,6 +158,9 @@ 'sources': [ 'src/crypto/evp/example_sign.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_hmac_test', @@ -135,6 +171,9 @@ 'sources': [ 'src/crypto/hmac/hmac_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_lhash_test', @@ -145,6 +184,9 @@ 'sources': [ 'src/crypto/lhash/lhash_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_md5_test', @@ -155,6 +197,9 @@ 'sources': [ 'src/crypto/md5/md5_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_gcm_test', @@ -165,6 +210,9 @@ 'sources': [ 'src/crypto/modes/gcm_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_pkcs12_test', @@ -175,6 +223,9 @@ 'sources': [ 'src/crypto/pkcs8/pkcs12_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_rsa_test', @@ -185,6 +236,9 @@ 'sources': [ 'src/crypto/rsa/rsa_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_sha1_test', @@ -195,6 +249,9 @@ 'sources': [ 'src/crypto/sha/sha1_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_pkcs7_test', @@ -205,6 +262,9 @@ 'sources': [ 'src/crypto/x509/pkcs7_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_pqueue_test', @@ -215,6 +275,9 @@ 'sources': [ 'src/ssl/pqueue/pqueue_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, { 'target_name': 'boringssl_ssl_test', @@ -225,6 +288,9 @@ 'sources': [ 'src/ssl/ssl_test.c', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], }, ], 'variables': { diff --git a/third_party/boringssl/update_gypi_and_asm.py b/third_party/boringssl/update_gypi_and_asm.py index 41196a5..58eb6f0 100644 --- a/third_party/boringssl/update_gypi_and_asm.py +++ b/third_party/boringssl/update_gypi_and_asm.py @@ -220,6 +220,9 @@ def main(): 'sources': [ '%s', ], + # TODO(davidben): Fix size_t truncations in BoringSSL. + # https://crbug.com/429039 + 'msvs_disabled_warnings': [ 4267, ], },\n""" % (test_name, test)) test_names.append(test_name) |