From: Antonio Larrosa Date: Mon, 6 Mar 2017 13:54:52 +0100 Subject: Check for multiplication overflow in sfconvert Checks that a multiplication doesn't overflow when calculating the buffer size, and if it overflows, reduce the buffer size instead of failing. This fixes the 00192-audiofile-signintoverflow-sfconvert case in #41 --- sfcommands/sfconvert.c | 34 ++++++++++++++++++++++++++++++++-- 1 file changed, 32 insertions(+), 2 deletions(-) diff --git a/sfcommands/sfconvert.c b/sfcommands/sfconvert.c index 80a1bc4..970a3e4 100644 --- a/sfcommands/sfconvert.c +++ b/sfcommands/sfconvert.c @@ -45,6 +45,33 @@ void printusage (void); void usageerror (void); bool copyaudiodata (AFfilehandle infile, AFfilehandle outfile, int trackid); +int firstBitSet(int x) +{ + int position=0; + while (x!=0) + { + x>>=1; + ++position; + } + return position; +} + +#ifndef __has_builtin +#define __has_builtin(x) 0 +#endif + +int multiplyCheckOverflow(int a, int b, int *result) +{ +#if (defined __GNUC__ && __GNUC__ >= 5) || ( __clang__ && __has_builtin(__builtin_mul_overflow)) + return __builtin_mul_overflow(a, b, result); +#else + if (firstBitSet(a)+firstBitSet(b)>31) // int is signed, so we can't use 32 bits + return true; + *result = a * b; + return false; +#endif +} + int main (int argc, char **argv) { if (argc == 2) @@ -323,8 +350,11 @@ bool copyaudiodata (AFfilehandle infile, AFfilehandle outfile, int trackid) { int frameSize = afGetVirtualFrameSize(infile, trackid, 1); - const int kBufferFrameCount = 65536; - void *buffer = malloc(kBufferFrameCount * frameSize); + int kBufferFrameCount = 65536; + int bufferSize; + while (multiplyCheckOverflow(kBufferFrameCount, frameSize, &bufferSize)) + kBufferFrameCount /= 2; + void *buffer = malloc(bufferSize); AFframecount totalFrames = afGetFrameCount(infile, AF_DEFAULT_TRACK); AFframecount totalFramesWritten = 0;
path: root/gnu/packages/kerberos.scm
AgeCommit message (Expand)Author
2020-10-08gnu: mit-krb5: Enable TLS support....Also enable more tests. * gnu/packages/kerberos.scm (mit-krb5)[native-inputs]: Add tcl. [inputs]: Add openssl. Maxim Cournoyer
2020-05-17gnu: heimdal: Update to 7.7.0....And fix the test suite failures as a side effect. * gnu/packages/kerberos.scm (heimdal): Update to 7.7.0. Maxim Cournoyer
2020-03-26gnu: mit-krb5: Supprt for the Hurd....* gnu/packages/patches/mit-krb5-hurd.patch: New file. * gnu/local.mk (dist_patch_DATA): Add it. * gnu/packages/kerberos.scm (mit-krb5): Use it. Jan Nieuwenhuizen
2020-03-14gnu: mit-krb5: Fix segfault in gsasl test suite....* gnu/packages/patches/mit-krb5-qualify-short-hostnames.patch: New file. * gnu/local.mk (dist_patch_DATA): Adjust accordingly. * gnu/packages/kerberos.scm (mit-krb5)[source](patches): New field. Marius Bakke
2020-03-10Merge branch 'master' into core-updatesMarius Bakke
2020-03-09gnu: mit-krb5: Use HTTPS home page....* gnu/packages/kerberos.scm (mit-krb5)[home-page]: Use HTTPS. Tobias Geerinckx-Rice
2020-02-15gnu: mit-krb5: Update to 1.18....* gnu/packages/kerberos.scm (mit-krb5): Update to 1.18. Marius Bakke
2020-01-17gnu: mit-krb5: Set localstatedir to /var....* gnu/packages/kerberos.scm (mit-krb5)[arguments]: Set localstatedir. Ricardo Wurmus
2020-01-08gnu: mit-krb5: Update to 1.17.1....* gnu/packages/kerberos.scm (mit-krb5): Update to 1.17.1. Marius Bakke