Skip to content

Fix CID-1518256 #10403

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.

Sign up for GitHub

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

Jump to bottom
Merged
moonchen merged 3 commits into from
Sep 14, 2023
Merged

Fix CID-1518256 #10403

moonchen merged 3 commits into from
Sep 14, 2023

Conversation

@moonchen
Copy link
Contributor

@moonchen moonchen commented Sep 12, 2023 •
edited
Loading

@moonchen
Fix CID-1518256
783629d 
* Read the return value from read() into a dummy variable.
@moonchen moonchen self-assigned this Sep 12, 2023
@moonchen moonchen added this to the 10.0.0 milestone Sep 12, 2023
@moonchen moonchen merged commit 8fcc9c3 into apache:master Sep 14, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
@zwoop
Merge remote-tracking branch 'asf/master'
868b9f6 
* asf/master:
  This drops the _stat suffix from some metrics (apache#10441)
  Fix CID-1518256 (apache#10403)
  Restore original metrics names, these were typos (apache#10440)
  Use the new Metrics module for core metrics (apache#10175)
  Fix use-after-free issue (apache#10399)
  Fixed differences between cmake rc files and autotools (apache#10408)
  Fix hwloc build (apache#10406)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bneradt bneradt bneradt approved these changes

Assignees

@moonchen moonchen

Labels

Coverity

Projects

None yet

Milestone

10.0.0

Development

Successfully merging this pull request may close these issues.

CID 1518256: Ignoring number of bytes read

2 participants

@moonchen @bneradt