Page MenuHomeFreeBSD

bsddialog: Correct type for bsddialog_total_progview
ClosedPublic

Authored by emaste on May 9 2025, 12:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 14, 3:12 AM
Unknown Object (File)
Mon, Oct 13, 5:32 AM
Unknown Object (File)
Sun, Oct 12, 12:15 PM
Unknown Object (File)
Fri, Oct 10, 10:02 AM
Unknown Object (File)
Thu, Oct 9, 8:43 AM
Unknown Object (File)
Fri, Oct 3, 5:50 PM
Unknown Object (File)
Thu, Oct 2, 4:48 AM
Unknown Object (File)
Wed, Oct 1, 6:27 AM
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.