Page MenuHomeFreeBSD

smbios: Add length sanity checking
AcceptedPublic

Authored by imp on Thu, Jun 27, 9:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 30, 10:29 AM
Unknown Object (File)
Fri, Jun 28, 5:24 PM
Subscribers
None

Details

Summary

D28743 was commited, reverted and then f689cb23b2782 landed before it
was recommitted. However, D28743 included an extra length check. Redo
that functionality so we check both the number of entries as well as the
length checks for wacky data.

Sponsored by: Netflix

Test Plan

I re-wrote this, but if people think it's too close (it did it completely different way, but the nature of C makes it a little similar), I'll add a co-authored-by.

Diff Detail

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