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, Sep 20, 9:33 PM

	Unknown Object (File)
	Sun, Sep 20, 9:32 PM

	Unknown Object (File)
	Sun, Sep 20, 6:00 PM

	Unknown Object (File)
	Fri, Sep 18, 6:27 AM

	Unknown Object (File)
	Fri, Sep 18, 1:14 AM

	Unknown Object (File)
	Sat, Sep 12, 3:57 AM

	Unknown Object (File)
	Thu, Sep 10, 10:43 AM

	Unknown Object (File)
	Aug 24 2026, 4:21 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