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

tool_operate: fix scanbuild compiler warning #10559

Closed
wants to merge 1 commit into from

Conversation

jay
Copy link
Member

@jay jay commented Feb 18, 2023

Prior to this change Azure CI scanbuild warned of a potential NULL pointer string passed to strtol when CURLDEBUG enabled, even though the way the code was written it wouldn't have happened.

Bug: 5479d991#r101159711
Reported-by: Marcel Raad

Closes #xxxx

Prior to this change Azure CI scanbuild warned of a potential NULL
pointer string passed to strtol when CURLDEBUG enabled, even though the
way the code was written it wouldn't have happened.

Bug: curl@5479d991#r101159711
Reported-by: Marcel Raad

Closes #xxxx
@jay jay added the CI Continuous Integration label Feb 18, 2023
jay referenced this pull request Feb 18, 2023
Using the CURL_BUFFERSIZE environment variable.

Closes #10532
@jay jay added the tidy-up label Feb 18, 2023
Copy link
Member

@MarcelRaad MarcelRaad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jay jay closed this in 41dfb7f Feb 19, 2023
@jay jay deleted the fix_azure_ci_warn branch February 19, 2023 00:04
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
Prior to this change Azure CI scanbuild warned of a potential NULL
pointer string passed to strtol when CURLDEBUG enabled, even though the
way the code was written it wouldn't have happened.

Bug: curl@5479d991#r101159711
Reported-by: Marcel Raad

Closes curl#10559
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration cmdline tool tidy-up
Development

Successfully merging this pull request may close these issues.

None yet

3 participants