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

Fixing low level and medium issues in static code analysis #167

Closed
wants to merge 1 commit into from
Closed

Fixing low level and medium issues in static code analysis #167

wants to merge 1 commit into from

Conversation

devath1x
Copy link

@devath1x devath1x commented Aug 22, 2024 •
edited
Loading

Fixing the medium and low level issues in static code analysis

Tracked-On: OAM-123193

@sysopenci
Copy link

Dependent pr program name is not compatable this pr's program name, for more details please check tracked_on

@sysopenci
Copy link

Dependent pr program name is not compatable this pr's program name, for more details please check tracked_on

@sysopenci
Copy link

Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added the Engineering Build Successful Engineering Build Successful label Aug 27, 2024
@devath1x devath1x changed the title [Coverity] Fix mediam and low impact issues Fixing low level and medium issues in static code analysis Aug 27, 2024
@devath1x
Fixing low level and medium issues in static code analysis
c20aaed 
Uninitialized and COPY_INSTED_OF_MOVE
issues are resolved in bluetooth interface

Tested:
BT on/off -> 20times
Connect phone, play music -> 10 times
HFP call -> 10 times
File transfer
CtsBluetoothTestCases

Tracked-On: OAM-123193
Signed-off-by: devath1x <[email protected]>
Signed-off-by: Gowtham Anandha Babu <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

bpanda24
bpanda24 approved these changes Aug 29, 2024
View reviewed changes
Copy link
Contributor

@bpanda24 bpanda24 left a comment

Choose a reason for hiding this comment

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

LGTM, all basic sanity checked

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Failed and removed Engineering Build Successful Engineering Build Successful labels Sep 10, 2024
@devath1x devath1x closed this Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bpanda24 bpanda24 bpanda24 approved these changes

@balajim001 balajim001 Awaiting requested review from balajim001

@feijiang1 feijiang1 Awaiting requested review from feijiang1

@gkdeepa gkdeepa Awaiting requested review from gkdeepa

@JeevakaPrabu JeevakaPrabu Awaiting requested review from JeevakaPrabu

Assignees
No one assigned
Labels
Engineering Build Failed Valid commit message
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@devath1x @sysopenci @bpanda24