Skip to content
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

Fix an UBSAN error #19243

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

zhihongliuus
Copy link

external/com_google_protobuf/src/google/protobuf/stubs/stringpiece.h:362:23: runtime error: null pointer passed as argument 1, which is declared to never be null
external/com_google_protobuf/src/google/protobuf/stubs/stringpiece.h:362:23: runtime error: null pointer passed as argument 2, which is declared to never be null

external/com_google_protobuf/src/google/protobuf/stubs/stringpiece.h:362:23: runtime error: null pointer passed as argument 1, which is declared to never be null
external/com_google_protobuf/src/google/protobuf/stubs/stringpiece.h:362:23: runtime error: null pointer passed as argument 2, which is declared to never be null
@zhihongliuus zhihongliuus requested a review from a team as a code owner November 13, 2024 00:38
@zhihongliuus zhihongliuus requested review from acozzette and removed request for a team November 13, 2024 00:38
Copy link

google-cla bot commented Nov 13, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@acozzette
Copy link
Member

Sorry but unfortunately we can't review this pull request until you sign the CLA.

Also I suspect the fix might not address the error you ran into, because we deleted our StringPiece code two years ago. You might find that the problem goes away on its own if you upgrade to a more recent protobuf version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants