fruit: disable useless size_t overflow check
authorMichael Tokarev <mjt@tls.msk.ru>
Thu, 17 Nov 2022 20:51:03 +0000 (23:51 +0300)
committerMichael Tokarev <mjt@tls.msk.ru>
Mon, 6 Jan 2025 18:20:31 +0000 (21:20 +0300)
commita5871772b8aef8d98c249f93a6fb7c67eb8b06f2
tree727fe285d23d8d4cae01f8523c54a4eef4e408e6
parenta6f4d641a5fb59c89dcdeba0696fe2010babffb5
fruit: disable useless size_t overflow check

Bug-Debian: http://bugs.debian.org/974868
Forwarded: https://gitlab.com/samba-team/samba/-/merge_requests/3872

As has been said several times in
https://bugzilla.samba.org/show_bug.cgi?id=13622 ,
the check 'bandsize > SIZE_MAX/nbands' is useless.  But it
is also wrong, in 2 ways: first, nbands might be 0 (when
no bands has been allocated yet), and second, there's no
point in comparing this with SIZE_MAX, since size_t on 32bit
platforms is a 32bit integer, while bandsize is off_t which
is 64bits (samba always enables LFS).

This check causes the module to fail when bandsize*nbands
exceeds 32bits, which has been reported for example at
https://bugs.debian.org/974868 .

Whole thing can't overflow because it is already guarded
by time_machine_max_size.  Or at the very least, by current
disk sizes... :)

Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Gbp-Pq: Name fruit-disable-useless-size_t-overflow-check.patch
source3/modules/vfs_fruit.c