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.

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

quic: address coverity warning #52824

Closed
wants to merge 1 commit into from
Closed

quic: address coverity warning #52824

wants to merge 1 commit into from

Conversation

mhdawson
Copy link
Member

@mhdawson mhdawson commented May 3, 2024

  • initiaze variable in similar way down in other constructors

@mhdawson
quic: address coverity warning
16342f4 
- initiaze variable in similar way down in other
  constructors

Signed-off-by: Michael Dawson <[email protected]>
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3. labels May 3, 2024
@mhdawson
Copy link
Member Author

mhdawson commented May 3, 2024

Coverity report

170QuicError::QuicError(const ngtcp2_ccerr* ptr)
171    : reason_(reinterpret_cast<const char*>(ptr->reason), ptr->reasonlen),
     	2. uninit_member: Non-static class member error_.type is not initialized in this constructor nor in any functions that it calls.
     	4. uninit_member: Non-static class member error_.error_code is not initialized in this constructor nor in any functions that it calls.
     	6. uninit_member: Non-static class member error_.frame_type is not initialized in this constructor nor in any functions that it calls.
     	8. uninit_member: Non-static class member error_.reason is not initialized in this constructor nor in any functions that it calls.
     	
CID 338953: (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
10. uninit_member: Non-static class member error_.reasonlen is not initialized in this constructor nor in any functions that it calls.
172      ptr_(ptr) {}
173

@mhdawson mhdawson added the request-ci Add this label to start a Jenkins CI on a PR. label May 6, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 6, 2024
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/58988/

@RedYetiDev RedYetiDev added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 6, 2024
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/58990/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/59044/

@mhdawson mhdawson added the request-ci Add this label to start a Jenkins CI on a PR. label May 9, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 9, 2024
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/59064/

mhdawson added a commit that referenced this pull request May 10, 2024
@mhdawson
quic: address coverity warning
fd1e66f 
- initiaze variable in similar way down in other
  constructors

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #52824
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@mhdawson
Copy link
Member Author

Landed in Landed in fd1e66f

@mhdawson mhdawson closed this May 10, 2024
targos pushed a commit that referenced this pull request May 11, 2024
@mhdawson @targos
quic: address coverity warning
619ac79 
- initiaze variable in similar way down in other
  constructors

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #52824
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@targos targos mentioned this pull request May 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jasnell jasnell jasnell approved these changes

@lpinca lpinca lpinca approved these changes

@anonrig anonrig anonrig approved these changes

@tniessen tniessen tniessen approved these changes

Assignees
No one assigned
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. quic Issues and PRs related to the QUIC implementation / HTTP/3.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@mhdawson @nodejs-github-bot @jasnell @lpinca @anonrig @tniessen @RedYetiDev