quic: address new coverity warning#48384
Conversation
mhdawson
commented
Jun 7, 2023
mhdawson
commented
- address coverity warning about unitialized variable
- address coverity warning about unitialized variable Signed-off-by: Michael Dawson <mdawson@devrus.com>
nodejs-github-bot
added
c++
debadree25
added
the
author ready
github-actions
Bot
removed
the
request-ci
| auto& state = BindingData::Get(env); | ||
| auto params = value.As<Object>(); | ||
| Options options; | ||
| Options options = Options(); |
There was a problem hiding this comment.
This seems odd. I might be missing something, but the explicit constructor call Options() should perform the same initialization as the implicit call in Options options, and assignment should not change anything either.
There was a problem hiding this comment.
@tniessen I'm not 100% sure this will fix the warning as I don't know that we can run coverity locally. I think other similar reports have been resolved by a similar fix through. Do you have an alternate suggestion?
There was a problem hiding this comment.
@tniessen I'll commit to looking to make sure it resolved the report after this lands. If it does not I'll figure out some other change that should. I'm thinking that is the easiest way to figure out if it does/does not unless you have another suggestion.
There was a problem hiding this comment.
@mhdawson Sorry, I missed the first ping. This still seems odd to me, especially given that coverity complains about options.transport_params.preferred_address_ipv4._M_payload._M_payload (that is, a field in a class that is not Options), but this PR has the necessary approvals to land and I won't block it.
There was a problem hiding this comment.
@tniessen you were correct. My change did not resolve the Coverity report. Will investigate further to better understand what is being reported.
- address coverity warning about unitialized variable Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: #48384 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
|
Landed in ac6f594 |
- address coverity warning about unitialized variable Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: nodejs#48384 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
- address coverity warning about unitialized variable Signed-off-by: Michael Dawson <mdawson@devrus.com> PR-URL: nodejs#48384 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
7 participants
