-
Notifications
You must be signed in to change notification settings - Fork 62
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 Static Analysis Issues for Sensor Module for IVI #124
Fix Static Analysis Issues for Sensor Module for IVI #124
Conversation
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
Android CI has completed Engineering Build for this issue, build is SUCCESS.To merge the changes please click on APPROVE FOR MERGE button in Android CI WEB |
e6be8a7
to
ea585da
Compare
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
Android CI has completed Engineering Build for this issue, build is SUCCESS.To merge the changes please click on APPROVE FOR MERGE button in Android CI WEB |
Below Static Analysis Issues will be solved with this change. +-------------------------------+ |Useless call | |Resource leak | |Dereference before null check | |Missing assignment operator | |Untrusted value as argument | |Uninitialized pointer field | |Uninitialized scalar field | |Uninitialized scalar variable | |AUTO_CAUSES_COPY | |Dead default in switch | |Uncaught exception | |Dead default in switch | |Out-of-bounds read | |Tainted-scalar | | +-------------------------------+ coverity fix for sensors Tracked-On: OAM-110040 Signed-off-by: Ranjan, Rajani <[email protected]>
ea585da
to
668d26f
Compare
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
Android CI has completed Engineering Build for this issue, build is SUCCESS.To merge the changes please click on APPROVE FOR MERGE button in Android CI WEB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failure when executing preintegration builder
FAILURE:CheckBug Bad comments/Bugs
More details, logs, and binaries: /absp/builders/celadon_smr0_master-preintegration/builds/1370
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR integrated into 'preintegration' builder
More details, logs, and binaries: /absp/builders/celadon_smr0_master-preintegration/builds/1371
Done by "Ranjan, Rajani" [email protected]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR integrated into 'premerge' builder
More details, logs, and binaries: /absp/builders/celadon_smr0_master-premerge/builds/732
Done by "Ranjan, Rajani" [email protected]
3d8c488
into
projectceladon:celadon/s/mr0/master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cactus finished building and testing your changes
It was requested for merge by "Ranjan, Rajani" [email protected]
so we merge it. Thanks for your contribution.
more details, logs, and binaries: /absp/builders/celadon_smr0_master-mergerequest/builds/749
Fix Static Analysis Issues for Sensor Module for IVI
Below Static Analysis Issues will be solved with this change.
+-------------------------------+
|Useless call |
|Resource leak |
|Dereference before null check |
|Missing assignment operator |
|Untrusted value as argument |
|Uninitialized pointer field |
|Uninitialized scalar field |
|Uninitialized scalar variable |
|AUTO_CAUSES_COPY |
|Dead default in switch |
|Uncaught exception |
|Dead default in switch |
|Out-of-bounds read |
+-------------------------------+
coverity fix for sensors
Tracked-On: OAM-110040
Signed-off-by: Ranjan, Rajani [email protected]