From d500ef3bb52d3989476e9ab41170ac138dcfd5f2 Mon Sep 17 00:00:00 2001 From: Shaolin Date: Tue, 7 Aug 2018 22:24:54 -0300 Subject: [PATCH] obs-outputs: Make rtmp packet alloc code path clearer On 64bit systems, this check will always evaluate to false due to SIZE_MAX type and triggers a compiler warning. This both makes it clearer that its only needed on 32bit system and clear the compiler warning. --- plugins/obs-outputs/librtmp/rtmp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/plugins/obs-outputs/librtmp/rtmp.c b/plugins/obs-outputs/librtmp/rtmp.c index cb2aa765f..71e7c1900 100644 --- a/plugins/obs-outputs/librtmp/rtmp.c +++ b/plugins/obs-outputs/librtmp/rtmp.c @@ -242,8 +242,10 @@ int RTMPPacket_Alloc(RTMPPacket *p, uint32_t nSize) { char *ptr; +#if ARCH_BITS == 32 if (nSize > SIZE_MAX - RTMP_MAX_HEADER_SIZE) return FALSE; +#endif ptr = calloc(1, nSize + RTMP_MAX_HEADER_SIZE); if (!ptr)