From: Jaroslav Kysela <perex@perex.cz>
Date: Sun, 9 Dec 2018 16:12:45 +0000 (+0100)
Subject: fix some problems detected by cppcheck, issue #5353
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11f6531a09bac850edefbd8df950173abbe3ad45;p=thirdparty%2Ftvheadend.git

fix some problems detected by cppcheck, issue #5353
---

diff --git a/src/descrambler/cccam.c b/src/descrambler/cccam.c
index cfb3b1fe8..28d44d456 100644
--- a/src/descrambler/cccam.c
+++ b/src/descrambler/cccam.c
@@ -333,6 +333,9 @@ cccam_decode_card_data_reply(cccam_t *cccam, uint8_t *msg)
   psa = nprov ? alloca(nprov * sizeof(uint8_t *)) : NULL;
   saa = nprov ? alloca(nprov * 8) : NULL;
 
+  if (pid == NULL || psa == NULL || saa == NULL)
+    return -ENOMEM;
+
   msg2 = msg + 25;
   memset(saa, 0, nprov * 8);
   for (i = 0; i < nprov; i++) {
diff --git a/src/prop.h b/src/prop.h
index d56100354..373942281 100644
--- a/src/prop.h
+++ b/src/prop.h
@@ -72,11 +72,11 @@ typedef enum {
  * min/max/step helpers
  */
 #define INTEXTRA_RANGE(min, max, step) \
-  ((1<<31)|(((step)&0x7f)<<24)|(((max)&0xfff)<<12)|((min)&0xfff))
+  ((1<<31U)|(((step)&0x7fU)<<24)|(((max)&0xfffU)<<12)|((min)&0xfffU))
 
-#define INTEXTRA_IS_RANGE(e) (((e) & (1<<31)) != 0)
+#define INTEXTRA_IS_RANGE(e) (((e) & (1<<31U)) != 0)
 #define INTEXTRA_IS_SPLIT(e) !INTEXTRA_IS_RANGE(e)
-#define INTEXTRA_GET_STEP(e) (((e)>>24)&0x7f)
+#define INTEXTRA_GET_STEP(e) (((e)>>24)&0x7fU)
 #define INTEXTRA_GET_MAX(e)  ((e)&(1<<23)?-(0x800-(((e)>>12)&0x7ff)):(((e)>>12)&0x7ff))
 #define INTEXTRA_GET_MIN(e)  ((e)&(1<<11)?-(0x800-((e)&0x7ff)):((e)&0x7ff))
 #define INTEXTRA_GET_UMAX(e) (((e)>>12)&0xfff)