Page MenuHomeFreeBSD
Differential D37547

aesni: Remove misleading array bounds for aesni_decryt_ecb.
ClosedPublic
Actions

Authored by jhb on Nov 29 2022, 7:57 PM.
Tags
None
Referenced Files
F102695952: D37547.diff
Fri, Nov 15, 11:45 PM
Unknown Object (File)
Fri, Nov 1, 4:56 PM
Unknown Object (File)
Sep 30 2024, 10:16 AM
Unknown Object (File)
Sep 24 2024, 6:10 PM
Unknown Object (File)
Sep 24 2024, 4:06 AM
Unknown Object (File)
Sep 18 2024, 4:29 AM
Unknown Object (File)
Sep 18 2024, 2:40 AM
Unknown Object (File)
Sep 17 2024, 7:52 AM
View All 56 Files
Subscribers
imp

Details

Reviewers
emaste
imp
Commits
rG22cb9d3f17c4: aesni: Remove misleading array bounds for aesni_decryt_ecb.
rGd256a06fe8bd: aesni: Remove misleading array bounds for aesni_decryt_ecb.
Summary

All the other functions used pointers for from/to instead of
fixed-size array parameters. More importantly, this function can
accept pointers to buffers of multiple blocks, not just a single
block.

Reported by: GCC -Warray-parameter

Diff Detail

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

Event Timeline

jhb created this revision.Nov 29 2022, 7:57 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 29 2022, 7:57 PM
jhb requested review of this revision.Nov 29 2022, 7:57 PM
jhb added a parent revision: D37546: net80211: Fix mismatches in array bounds..
Harbormaster completed remote builds in B48548: Diff 113634.Nov 29 2022, 7:57 PM
jhb added a child revision: D37548: ipfilter: Fix mismatch in array bounds..Nov 29 2022, 7:57 PM
jhb added a comment.Nov 29 2022, 8:03 PM

This one was actually not just cosmetic but a bug, if only we actually used ECB mode for anything (we don't).

imp accepted this revision.Nov 29 2022, 8:21 PM
This revision is now accepted and ready to land.Nov 29 2022, 8:21 PM
emaste accepted this revision.Dec 2 2022, 10:49 PM
Closed by commit rGd256a06fe8bd: aesni: Remove misleading array bounds for aesni_decryt_ecb. (authored by jhb). · Explain WhyDec 7 2022, 8:34 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGd256a06fe8bd: aesni: Remove misleading array bounds for aesni_decryt_ecb..
jhb added a commit: rG22cb9d3f17c4: aesni: Remove misleading array bounds for aesni_decryt_ecb..May 3 2023, 12:30 AM

Revision Contents
Changeset List

PathSize
sys/
crypto/
aesni/
2 lines

Diff 113634

View Options

sys/crypto/aesni/aesni_wrap.c

Loading...