Differential D35382

crypto: Validate return values from CRYPTODEV_PROCESS()
ClosedPublic
Actions

Authored by markj on Jun 1 2022, 9:24 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 9, 2:33 AM

	Unknown Object (File)
	Sat, Nov 8, 6:05 PM

	Unknown Object (File)
	Wed, Oct 29, 7:34 PM

	Unknown Object (File)
	Tue, Oct 28, 2:46 AM

	Unknown Object (File)
	Mon, Oct 27, 12:21 AM

	Unknown Object (File)
	Sun, Oct 26, 10:24 PM

	Unknown Object (File)
	Sun, Oct 26, 10:21 PM

	Unknown Object (File)
	Sat, Oct 25, 2:39 PM

Subscribers

Details

Reviewers

jhb
kp
mav

Commits

rG99df914899f5: crypto: Validate return values from CRYPTODEV_PROCESS()

Summary

Errors are always handled by the completion callback, so we should check
that they're not also passed back to the caller.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jun 1 2022, 9:24 PM

Herald added subscribers: jmg, imp. · View Herald TranscriptJun 1 2022, 9:24 PM

markj requested review of this revision.Jun 1 2022, 9:24 PM

Harbormaster completed remote builds in B45810: Diff 106546.Jun 1 2022, 9:24 PM

markj added a child revision: D35383: crypto: Remove the return value from crypto_dispatch(_async)().Jun 1 2022, 9:32 PM

kp accepted this revision.Jun 1 2022, 9:41 PM

This revision is now accepted and ready to land.Jun 1 2022, 9:41 PM

kp mentioned this in D35383: crypto: Remove the return value from crypto_dispatch(_async)().Jun 2 2022, 8:14 AM

jhb accepted this revision.Jun 29 2022, 9:45 PM

jhb added a reviewer: mav.

mav accepted this revision.Jun 30 2022, 3:03 AM

Closed by commit rG99df914899f5: crypto: Validate return values from CRYPTODEV_PROCESS() (authored by markj). · Explain WhyJul 1 2022, 3:31 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG99df914899f5: crypto: Validate return values from CRYPTODEV_PROCESS().

Revision Contents
Changeset List

Path

Size

sys/

opencrypto/

13 lines

Diff 107665

sys/opencrypto/crypto.c

Loading...