Page MenuHomeFreeBSD
Differential D3191

Catch uid and gid to the correct type for display to solve segfault in ls(1) on 32bit arches
ClosedPublic
Actions

Authored by allanjude on Jul 24 2015, 7:09 PM.
Tags
None
Referenced Files
F106651504: D3191.diff
Fri, Jan 3, 9:51 AM
Unknown Object (File)
Oct 18 2024, 8:39 AM
Unknown Object (File)
Oct 2 2024, 7:54 AM
Unknown Object (File)
Sep 22 2024, 4:04 AM
Unknown Object (File)
Sep 17 2024, 1:38 AM
Unknown Object (File)
Sep 1 2024, 7:13 AM
Unknown Object (File)
Aug 28 2024, 9:39 PM
Unknown Object (File)
Aug 18 2024, 10:51 PM
View All 70 Files
Subscribers
antoine
emaste
imp
ngie

Details

Reviewers
dim
eadler
ngie
marcel
Commits
rS285857: Cast uid and gid to the correct type for display to solve segfault in ls(1)…
Summary

Fix error introduced in rS285734 that causes segfault on 'ls -n' in i386

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

allanjude updated this revision to Diff 7273.Jul 24 2015, 7:09 PM
allanjude retitled this revision from to Catch uid and gid to the correct type for display to solve segfault in ls(1) on 32bit arches.
allanjude updated this object.
allanjude edited the test plan for this revision. (Show Details)
allanjude added reviewers: eadler, dim, marcel.
allanjude added a subscriber: antoine.
Herald added a subscriber: imp. · View Herald TranscriptJul 24 2015, 7:09 PM
emaste added a subscriber: emaste.Jul 24 2015, 7:14 PM
ngie accepted this revision.Jul 24 2015, 7:24 PM
ngie added a reviewer: ngie.
ngie added a subscriber: ngie.
ngie added inline comments.
bin/ls/print.c
195 ↗(On Diff #7273)

Please remove the space between the casts and the casted values

This revision is now accepted and ready to land.Jul 24 2015, 7:24 PM
allanjude updated this revision to Diff 7274.Jul 24 2015, 7:29 PM
allanjude edited edge metadata.

Fix another error detected by clang with __printflike applied to xo_emit

Remove space between casts and values

This revision now requires review to proceed.Jul 24 2015, 7:29 PM
dim accepted this revision.Jul 24 2015, 8:11 PM
dim edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jul 24 2015, 8:11 PM
Closed by commit rS285857: Cast uid and gid to the correct type for display to solve segfault in ls(1)… (authored by allanjude). · Explain WhyJul 24 2015, 8:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
bin/
ls/
4 lines

Diff 7275

View Options

head/bin/ls/print.c

Loading...