Do not destroy input buffer of the OF_getencprop() function on error
ClosedPublic
Actions

Authored by zbb on Jan 25 2016, 12:31 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, May 25, 6:18 PM

	Unknown Object (File)
	May 14 2025, 8:14 PM

	Unknown Object (File)
	May 4 2025, 2:23 PM

	Unknown Object (File)
	Apr 9 2025, 11:10 AM

	Unknown Object (File)
	Apr 9 2025, 10:50 AM

	Unknown Object (File)
	Apr 9 2025, 8:04 AM

	Unknown Object (File)
	Apr 1 2025, 8:31 AM

	Unknown Object (File)
	Feb 5 2025, 7:04 PM

Subscribers

Details

Reviewers

mmel
• ian

Summary

Currently when the OF_getprop() function returns with error,
the caller (OF_getencprop()) still changes the buffer endiannes.
This may destroy the default value passed in the input buffer if
used on a Little Endian platform.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

zbb updated this revision to Diff 12680.Jan 25 2016, 12:31 PM

zbb retitled this revision from to Do not destroy input buffer of the OF_getencprop() function on error.

zbb updated this object.

zbb edited the test plan for this revision. (Show Details)

zbb added a reviewer: • ian.

zbb set the repository for this revision to rS FreeBSD src repository - subversion.

zbb added subscribers: ARM, PowerPC.

Herald added a subscriber: imp. · View Herald TranscriptJan 25 2016, 12:31 PM

mmel accepted this revision.Jan 25 2016, 12:33 PM

mmel added a reviewer: mmel.

This revision is now accepted and ready to land.Jan 25 2016, 12:33 PM

Committed to r294729

Revision Contents
Changeset List

Path

Size

sys/

dev/

ofw/

openfirm.c

3 lines

Diff 12680

View Options

Do not destroy input buffer of the OF_getencprop() function on errorClosedPublicActions