New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[php ext] build failure with recent GCC #10843

Closed
remicollet opened this Issue Apr 25, 2017 · 9 comments

Comments

Projects
None yet
7 participants
@remicollet

remicollet commented Apr 25, 2017

ON Fedora 26, using GCC 7.0

In file included from /usr/include/string.h:639:0,
                 from /builddir/build/BUILD/php-pecl-grpc-1.2.0/NTS/third_party/boringssl/crypto/asn1/a_bitstr.c:59:
In function 'memcpy',
    inlined from 'i2c_ASN1_BIT_STRING' at /builddir/build/BUILD/php-pecl-grpc-1.2.0/NTS/third_party/boringssl/crypto/asn1/a_bitstr.c:118:5:
/usr/include/bits/string3.h:53:10: error: '__builtin_memcpy': specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Werror=stringop-overflow=]
   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/builddir/build/BUILD/php-pecl-grpc-1.2.0/NTS/third_party/boringssl/crypto/asn1/a_bitstr.c: At top level:
cc1: error: unrecognized command line option '-Wno-parentheses-equality' [-Werror]
cc1: all warnings being treated as errors
@nicolasnoble

This comment has been minimized.

Show comment
Hide comment
@nicolasnoble

nicolasnoble Apr 25, 2017

Contributor

Adding @jboeuf on behalf of @agl - that seems there's an interesting memcpy happening there.

Contributor

nicolasnoble commented Apr 25, 2017

Adding @jboeuf on behalf of @agl - that seems there's an interesting memcpy happening there.

@jboeuf

This comment has been minimized.

Show comment
Hide comment
@jboeuf

jboeuf Apr 25, 2017

Contributor

This looks like a compilation error, right? I'm a bit confused by these numbers and where they're coming from. Do we have more context?
Adding @murgatroid99 @stanley-cheung for gRPC PHP.

Contributor

jboeuf commented Apr 25, 2017

This looks like a compilation error, right? I'm a bit confused by these numbers and where they're coming from. Do we have more context?
Adding @murgatroid99 @stanley-cheung for gRPC PHP.

@remicollet

This comment has been minimized.

Show comment
Hide comment
@remicollet

remicollet Apr 26, 2017

This looks like a compilation error, right?

Yes.

Do we have more context?

Sorry, no, simple build (on Fedora 26)

remicollet commented Apr 26, 2017

This looks like a compilation error, right?

Yes.

Do we have more context?

Sorry, no, simple build (on Fedora 26)

@jboeuf

This comment has been minimized.

Show comment
Hide comment
@jboeuf

jboeuf Apr 26, 2017

Contributor

Thanks @remicollet

@nicolasnoble did you try to compile just our version of boringssl on fedora 26 with gcc 7.0? It doesn't look like a gRPC bug to me. Note that our version of boring doesn't include this fix google/boringssl@17cf2cb which may address the problem.

Contributor

jboeuf commented Apr 26, 2017

Thanks @remicollet

@nicolasnoble did you try to compile just our version of boringssl on fedora 26 with gcc 7.0? It doesn't look like a gRPC bug to me. Note that our version of boring doesn't include this fix google/boringssl@17cf2cb which may address the problem.

@stanley-cheung

This comment has been minimized.

Show comment
Hide comment
@stanley-cheung

stanley-cheung Apr 26, 2017

Contributor

I am trying to reproduce this. Trying to find whether there's an easier way to run Fedora 26 Alpha. There don't seem to be a docker image.

Contributor

stanley-cheung commented Apr 26, 2017

I am trying to reproduce this. Trying to find whether there's an easier way to run Fedora 26 Alpha. There don't seem to be a docker image.

@choleraehyq

This comment has been minimized.

Show comment
Hide comment
@choleraehyq

choleraehyq Jul 2, 2017

Hi all, I also met this error when building TiKV.

/home/cholerae/rustcode/tikv/target/release/build/grpc-sys-08652de717d3e248/out/grpc/third_party/boringssl/crypto/asn1/a_bitstr.c: In function ‘i2c_ASN1_BIT_STRING’:
/home/cholerae/rustcode/tikv/target/release/build/grpc-sys-08652de717d3e248/out/grpc/third_party/boringssl/crypto/asn1/a_bitstr.c:118:5: error: ‘memcpy’: specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Werror=stringop-overflow=]
     memcpy(p, d, len);
     ^~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

They are using grpc 1.4.0.
Fedora 26, gcc 7.1.1.

choleraehyq commented Jul 2, 2017

Hi all, I also met this error when building TiKV.

/home/cholerae/rustcode/tikv/target/release/build/grpc-sys-08652de717d3e248/out/grpc/third_party/boringssl/crypto/asn1/a_bitstr.c: In function ‘i2c_ASN1_BIT_STRING’:
/home/cholerae/rustcode/tikv/target/release/build/grpc-sys-08652de717d3e248/out/grpc/third_party/boringssl/crypto/asn1/a_bitstr.c:118:5: error: ‘memcpy’: specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Werror=stringop-overflow=]
     memcpy(p, d, len);
     ^~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

They are using grpc 1.4.0.
Fedora 26, gcc 7.1.1.

@remicollet

This comment has been minimized.

Show comment
Hide comment
@remicollet

remicollet Jul 2, 2017

@choleraehyq on Fedora, try my repository ;)
https://rpms.remirepo.net/

Everything about PHP should be there (including this project)

remicollet commented Jul 2, 2017

@choleraehyq on Fedora, try my repository ;)
https://rpms.remirepo.net/

Everything about PHP should be there (including this project)

@jiangtaoli2016

This comment has been minimized.

Show comment
Hide comment
@jiangtaoli2016

jiangtaoli2016 May 16, 2018

Contributor

Is this still a problem? @remicollet

Contributor

jiangtaoli2016 commented May 16, 2018

Is this still a problem? @remicollet

@remicollet

This comment has been minimized.

Show comment
Hide comment
@remicollet

remicollet May 23, 2018

fixed for some time now

remicollet commented May 23, 2018

fixed for some time now

@remicollet remicollet closed this May 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment