Don't run strcmp() against strings stored in user memory.
ClosedPublic
Actions

Authored by jhb on Apr 24 2020, 11:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Oct 27, 6:57 AM

	Unknown Object (File)
	Mon, Oct 27, 5:47 AM

	Unknown Object (File)
	Sun, Oct 26, 7:58 PM

	Unknown Object (File)
	Sun, Oct 26, 7:57 PM

	Unknown Object (File)
	Sun, Oct 26, 7:56 PM

	Unknown Object (File)
	Sun, Oct 26, 2:12 PM

	Unknown Object (File)
	Sep 16 2025, 2:31 AM

	Unknown Object (File)
	Sep 5 2025, 8:22 PM

View All 74 Files

Subscribers

imp

Details

Reviewers

brooks
kib

Commits

rS360388: Don't run strcmp() against strings stored in user memory.

Summary

Instead, copy the strings into a temporary buffer on the stack and
run strcmp on the copies.

Test Plan

compile tested only

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Apr 24 2020, 11:50 PM

Harbormaster completed remote builds in B30716: Diff 70971.Apr 24 2020, 11:50 PM

kib accepted this revision.Apr 25 2020, 12:39 PM

This revision is now accepted and ready to land.Apr 25 2020, 12:39 PM

brooks accepted this revision.Apr 27 2020, 5:18 PM

Closed by commit rS360388: Don't run strcmp() against strings stored in user memory. (authored by jhb). · Explain WhyApr 27 2020, 6:04 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS360388: Don't run strcmp() against strings stored in user memory..

Herald added a subscriber: imp. · View Herald TranscriptApr 27 2020, 6:04 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

iscsi_initiator/

isc_subr.c

17 lines

Diff 71071

View Options

head/sys/dev/iscsi_initiator/isc_subr.c

Don't run strcmp() against strings stored in user memory.ClosedPublicActions