Page MenuHomeFreeBSD
Differential D30300

pms(4): clear CCBs allocated on the stack
ClosedPublic
Actions

Authored by trasz on May 16 2021, 8:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 9:48 PM
Unknown Object (File)
Sun, May 12, 3:33 AM
Unknown Object (File)
Sun, May 12, 1:34 AM
Unknown Object (File)
Sat, May 11, 6:12 PM
Unknown Object (File)
Fri, May 10, 8:19 PM
Unknown Object (File)
Apr 10 2024, 2:42 AM
Unknown Object (File)
Apr 9 2024, 10:00 PM
Unknown Object (File)
Feb 17 2024, 5:05 AM
View All 26 Files
Subscribers
imp
NetApp

Details

Reviewers
mav
imp
Commits
rGd39aac796bd2: pms(4): clear CCBs allocated on the stack

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 39226
Build 36115: arc lint + arc unit

Event Timeline

trasz created this revision.May 16 2021, 8:32 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 16 2021, 8:32 PM
trasz requested review of this revision.May 16 2021, 8:32 PM
Harbormaster completed remote builds in B39226: Diff 89313.May 16 2021, 8:32 PM
imp accepted this revision.May 16 2021, 10:45 PM
This revision is now accepted and ready to land.May 16 2021, 10:45 PM
trasz added a comment.May 17 2021, 8:38 AM

(Tinderboxed.)

allanjude added a subscriber: NetApp.May 19 2021, 4:57 PM
imp added a comment.May 20 2021, 2:16 AM

Picking a random one of these to reply to
https://reviews.freebsd.org/D30352
has an alternative function that I think is safer that we should consider.

imp added a comment.May 20 2021, 11:24 PM
In D30300#681823, @imp wrote:

Picking a random one of these to reply to
https://reviews.freebsd.org/D30352
has an alternative function that I think is safer that we should consider.

I've talked with trasz, and he'll proceed with his commits and I'll do this refinement at some later date.

Closed by commit rGd39aac796bd2: pms(4): clear CCBs allocated on the stack (authored by trasz). · Explain WhyMay 21 2021, 6:38 AM
This revision was automatically updated to reflect the committed changes.
trasz added a commit: rGd39aac796bd2: pms(4): clear CCBs allocated on the stack.

Revision Contents
Changeset List

PathSize
sys/
dev/
pms/
freebsd/
driver/
ini/
src/
3 lines

Diff 89313

View Options

sys/dev/pms/freebsd/driver/ini/src/agtiapi.c

Loading...