Check the return value from utf8_to_ucs2 instead of whether or not uv is NULL. That's more correct and doesn't depend on the error behavior of utf8_to_ucs2. In practice, we'll never see this though since we pass utf8_to_ucs2 a well formed string.
ClosedPublic
Actions

Authored by imp on Jan 15 2018, 5:03 PM.

Tags

None

Referenced Files

	F106416179: D13918.diff
	Mon, Dec 30, 9:25 AM

	Unknown Object (File)
	Sat, Dec 21, 8:06 AM

	Unknown Object (File)
	Nov 27 2024, 10:50 PM

	Unknown Object (File)
	Nov 27 2024, 5:53 AM

	Unknown Object (File)
	Nov 24 2024, 11:42 AM

	Unknown Object (File)
	Nov 16 2024, 10:06 PM

	Unknown Object (File)
	Nov 11 2024, 2:21 AM

	Unknown Object (File)
	Nov 11 2024, 1:54 AM

View All 50 Files

Subscribers

None

Details

Reviewers

andrew
tsoome

Commits

rS328030: Check the return value from utf8_to_ucs2 instead of whether or not uv

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

imp created this revision.Jan 15 2018, 5:03 PM

Harbormaster completed remote builds in B14385: Diff 37976.Jan 15 2018, 5:03 PM

imp added reviewers: andrew, tsoome.Jan 15 2018, 5:03 PM

tsoome added inline comments.Jan 15 2018, 7:58 PM

stand/efi/boot1/boot1.c
105 ↗	(On Diff #37976)	If we check the return value below, we do not need to set NULL here, except if the compiler will trigger may be used uninitialized warning...

imp added inline comments.Jan 15 2018, 10:09 PM

stand/efi/boot1/boot1.c
105 ↗	(On Diff #37976)	We do, because we want utf8_to_ucs2 to allocate space.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 15 2018, 10:17 PM

Closed by commit rS328030: Check the return value from utf8_to_ucs2 instead of whether or not uv (authored by imp). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

stand/

efi/

boot1/

boot1.c

3 lines

Diff 38001

View Options

head/stand/efi/boot1/boot1.c

Check the return value from utf8_to_ucs2 instead of whether or not uv is NULL. That's more correct and doesn't depend on the error behavior of utf8_to_ucs2. In practice, we'll never see this though since we pass utf8_to_ucs2 a well formed string.ClosedPublicActions