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.

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

coverity: fix coverity issue report in crashlog #30

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

bhe4
Copy link
Contributor

@bhe4 bhe4 commented Jan 12, 2025

  1. fix the resource leak in check_aplogs_tobackup
  2. fix the Memory - corruptions in add_section
  3. fix Uninitialized variables in enable_watchdog
  4. fix the resource leak in find_system_last_kmsg
  5. fix the Uninitialized variables in do_last_kmsg_copy

Tests Done:

  1. Build and boot
  2. crashlog function works

Tracked-On: OAM-129199

1. fix the resource leak in check_aplogs_tobackup
2. fix the Memory - corruptions in add_section
3. fix Uninitialized variables in enable_watchdog
4. fix the resource leak in find_system_last_kmsg
5. fix the Uninitialized variables in do_last_kmsg_copy

Tests Done:
1. Build and boot
2. crashlog function works

Tracked-On: OAM-129199
Signed-off-by: Bo He <[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.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Not Started Engineering Build Not Started labels Jan 12, 2025
@sysopenci
Copy link

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

@sysopenci sysopenci merged commit bc39509 into projectceladon:master Jan 13, 2025
21 checks passed
@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon-merge/933

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants