Page MenuHomeFreeBSD
Differential D26534

Misc compiler warning fixes in lib/libc
ClosedPublic
Actions

Authored by asomers on Sep 23 2020, 6:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 6, 10:07 PM
Unknown Object (File)
Wed, Nov 5, 7:57 PM
Unknown Object (File)
Sat, Nov 1, 3:08 AM
Unknown Object (File)
Tue, Oct 28, 6:43 AM
Unknown Object (File)
Mon, Oct 20, 10:59 AM
Unknown Object (File)
Sun, Oct 19, 11:56 AM
Unknown Object (File)
Sat, Oct 18, 4:34 PM
Unknown Object (File)
Sat, Oct 18, 5:37 AM
View All 94 Files
Subscribers
imp

Details

Reviewers
kevans
Commits
rS366207: Misc compiler warning fixes in lib/libc
Summary

Misc compiler warning fixes in lib/libc

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 33776
Build 30995: arc lint + arc unit

Event Timeline

asomers created this revision.Sep 23 2020, 6:12 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 23 2020, 6:12 PM
asomers requested review of this revision.Sep 23 2020, 6:12 PM
Harbormaster completed remote builds in B33776: Diff 77447.Sep 23 2020, 6:12 PM
imp added inline comments.Sep 23 2020, 7:42 PM
lib/libc/gen/auxv.c
270

Shouldn't we declare buflen to be a size_t instead?

asomers added inline comments.Sep 23 2020, 8:24 PM
lib/libc/gen/auxv.c
270

Can't, because _elf_aux_info must have the same signature as elf_aux_info

imp added inline comments.Sep 23 2020, 11:15 PM
lib/libc/gen/auxv.c
270

Then maybe we should widen buflen to size_t instead of narrowing the return value of strlcpy.

imp added a comment.Sep 23 2020, 11:19 PM

btw, the rest of the changes look totally fine. Sorry to be so nitpicky over this one...

lib/libc/gen/auxv.c
270

and do we need to make sure buflen isn't less than zero?

asomers updated this revision to Diff 77537.Sep 25 2020, 7:04 PM
asomers marked 3 inline comments as done.

Handle buflen more carefully in _elf_aux_info

Harbormaster completed remote builds in B33818: Diff 77537.Sep 25 2020, 7:04 PM
kevans accepted this revision.Sep 25 2020, 7:11 PM
kevans added inline comments.
lib/libc/gen/auxv.c
270

I would've perhaps opted to just widen it for this one comparison after checking that it's not negative above, but I think it's fine either way.

This revision is now accepted and ready to land.Sep 25 2020, 7:11 PM
Closed by commit rS366207: Misc compiler warning fixes in lib/libc (authored by asomers). · Explain WhySep 27 2020, 10:27 PM
This revision was automatically updated to reflect the committed changes.
asomers added a commit: rS366207: Misc compiler warning fixes in lib/libc.

Revision Contents
Changeset List

PathSize
lib/
libc/
gen/
2 lines
3 lines
5 lines
2 lines
5 lines
3 lines
1 line

Diff 77447

View Options

lib/libc/gen/auxv.c

Loading...
View Options

lib/libc/gen/basename_compat.c

Loading...
View Options

lib/libc/gen/crypt.c

Loading...
View Options

lib/libc/gen/dirname_compat.c

Loading...
View Options

lib/libc/gen/fts-compat.c

Loading...
View Options

lib/libc/gen/ftw-compat11.c

Loading...
View Options

lib/libc/gen/getentropy.c

Loading...