Page MenuHomeFreeBSD

x86/ucode: fix gcc uninitialised warning
AcceptedPublic

Authored by adrian on Feb 22 2026, 4:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 8, 9:51 AM
Unknown Object (File)
Mon, Apr 6, 2:31 PM
Unknown Object (File)
Sun, Apr 5, 1:10 PM
Unknown Object (File)
Sat, Apr 4, 10:18 PM
Unknown Object (File)
Thu, Mar 19, 7:22 AM
Unknown Object (File)
Mar 18 2026, 4:09 AM
Unknown Object (File)
Mar 2 2026, 11:58 PM
Unknown Object (File)
Feb 26 2026, 2:05 AM
Subscribers

Details

Reviewers
emaste
Summary

This fixes the case where selected_size is never set to anything
in the loop. Whilst here, also set selected_fw to NULL so the case
of "no firmware" correctly sets everything to NULL/0.

  --- ucode_subr.o ---
  /workspace/src/sys/x86/x86/ucode_subr.c: In function 'ucode_amd_find':
  /workspace/src/sys/x86/x86/ucode_subr.c:237:25: warning: 'selected_size' may be used uninitialized [-Wmaybe-uninitialized]
    237 |         *selected_sizep = selected_size;
        |         ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
/  workspace/src/sys/x86/x86/ucode_subr.c:105:16: note: 'selected_size' was declared here
    105 |         size_t selected_size;
        |                ^~~~~~~~~~~~~

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70879
Build 67762: arc lint + arc unit