Page MenuHomeFreeBSD

bsddialog: Correct type for bsddialog_total_progview
ClosedPublic

Authored by emaste on May 9 2025, 12:41 AM.
Tags
None
Referenced Files
F158671616: D50261.diff
Thu, Jun 4, 1:21 PM
Unknown Object (File)
Thu, May 28, 10:56 PM
Unknown Object (File)
Thu, May 28, 8:34 PM
Unknown Object (File)
Tue, May 19, 4:47 AM
Unknown Object (File)
Mon, May 18, 6:35 PM
Unknown Object (File)
Mon, May 18, 1:06 PM
Unknown Object (File)
Sun, May 17, 7:18 AM
Unknown Object (File)
Fri, May 15, 2:28 PM
Subscribers

Details

Summary
It was an int, but printed with %lli format.  Although it would be
reasonable to use a 32-bit int here (i.e., changing the printf format
instead) this matches what was done in bsddialog upstream (and now in
main).

PR:             286655

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste created this revision.
emaste added a reviewer: jhb.

OK, I remember the problem described in PR286655, I found it testing the installer on a 32bit (cheap) raspberry and watching a video on youtube (2 years ago). Of course "num extracted bytes" > "max int", I obseverd things like "-676330 files read" (negative num).
I fixed like in this review and now it is in main/CURRENT: I kept %lli because previously libdpv printed a %lli and I changed int to long long int.

This revision is now accepted and ready to land.May 9 2025, 4:52 PM

I fixed like in this review and now it is in main/CURRENT

Yeah; after 14.3 is out I think we should MFC the bsddialog updates. For now I just wanted a minimal change.