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)
	Tue, Oct 14, 5:22 AM

	Unknown Object (File)
	Mon, Sep 22, 2:07 AM

	Unknown Object (File)
	Aug 24 2025, 11:51 PM

	Unknown Object (File)
	Aug 3 2025, 6:08 PM

	Unknown Object (File)
	Aug 2 2025, 3:40 AM

	Unknown Object (File)
	Jul 27 2025, 11:02 PM

	Unknown Object (File)
	Jun 23 2025, 6:24 AM

	Unknown Object (File)
	Jun 22 2025, 7:59 AM

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