Page MenuHomeFreeBSD
Differential D13918

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
F167106694: D13918.diff
Wed, Aug 19, 3:38 AM
F167063825: D13918.diff
Tue, Aug 18, 8:26 PM
Unknown Object (File)
Thu, Aug 13, 6:20 PM
Unknown Object (File)
Tue, Aug 11, 6:23 PM
Unknown Object (File)
Tue, Aug 11, 4:15 PM
Unknown Object (File)
Sun, Aug 9, 11:14 AM
Unknown Object (File)
Fri, Aug 7, 7:43 AM
Unknown Object (File)
Thu, Aug 6, 4:30 PM
View All 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

PathSize
head/
stand/
efi/
boot1/
3 lines

Diff 38001

View Options

head/stand/efi/boot1/boot1.c

Loading...