Skip to content

Fix uninitialised read in apmlActiveMonitor#91

Open
illiav-j wants to merge 1 commit intoAMDESE:mainfrom
illiav-j:fix-uninit-read
Open

Fix uninitialised read in apmlActiveMonitor#91
illiav-j wants to merge 1 commit intoAMDESE:mainfrom
illiav-j:fix-uninit-read

Conversation

@illiav-j
Copy link
Copy Markdown

Fixes the error when trying to build on the latest OpenBMC:

| ../git/src/main.cpp: In function 'apmlActiveMonitor()':
| ../git/src/main.cpp:693:16: error: 'ret' is used uninitialized [-Werror=uninitialized]
|   693 |     while (ret != OOB_SUCCESS)
|       |            ~~~~^~~~~~~~~~~~~~
| ../git/src/main.cpp:689:18: note: 'ret' was declared here
|   689 |     oob_status_t ret;
|       |                  ^~~

This change has not been tested.

@ojayanth
Copy link
Copy Markdown

@abinayaddhandapani Can you please add the requeste changes in next refersh?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants