Avoid a dubious assignment to bio_data in aio_qbio().
ClosedPublic
Actions

Authored by jhb on Sep 29 2020, 9:14 PM.

Tags

None

Referenced Files

	F154811797: D26595.diff
	Wed, Apr 29, 12:22 PM

	F154588651: D26595.id77693.diff
	Tue, Apr 28, 11:57 PM

	Unknown Object (File)
	Tue, Apr 28, 10:33 AM

	Unknown Object (File)
	Tue, Apr 28, 4:46 AM

	Unknown Object (File)
	Mon, Apr 27, 5:57 AM

	Unknown Object (File)
	Fri, Apr 24, 6:48 AM

	Unknown Object (File)
	Tue, Apr 14, 1:08 PM

	Unknown Object (File)
	Tue, Apr 14, 8:45 AM

View All 88 Files

Subscribers

imp

Details

Reviewers

kib

Commits

rS366296: Avoid a dubious assignment to bio_data in aio_qbio().

Summary

A user pointer is not a suitable value for bio_data and the next block
of code always overwrites bio_data anyway. Just use cb->aio_buf
directly in the call to vm_fault_quick_hold_pages().

Test Plan

kyua test -k /usr/tests/sys/aio/Kyuafile

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33886
Build 31093: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Sep 29 2020, 9:14 PM

jhb created this revision.

Harbormaster completed remote builds in B33886: Diff 77652.Sep 29 2020, 9:14 PM

kib accepted this revision.Sep 30 2020, 3:22 AM

This revision is now accepted and ready to land.Sep 30 2020, 3:22 AM

Closed by commit rS366296: Avoid a dubious assignment to bio_data in aio_qbio(). (authored by jhb). · Explain WhySep 30 2020, 5:49 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS366296: Avoid a dubious assignment to bio_data in aio_qbio()..

Herald added a subscriber: imp. · View Herald TranscriptSep 30 2020, 5:49 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

vfs_aio.c

3 lines

Diff 77652

View Options

Avoid a dubious assignment to bio_data in aio_qbio().ClosedPublicActions