From bbc5c0e4ac7b85fd38fb3cc372b4b90242a0a107 Mon Sep 17 00:00:00 2001 From: "pr.jung" Date: Wed, 23 Mar 2016 19:22:33 +0900 Subject: [PATCH] Check argument first on transmit api Change-Id: Iecfcc15a5ee2e34b114fc61f60496a3f689abfe6 Signed-off-by: pr.jung --- src/ir.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/ir.c b/src/ir.c index 5b744cb..bc7b82e 100644 --- a/src/ir.c +++ b/src/ir.c @@ -66,6 +66,13 @@ int device_ir_transmit(int carrier_frequency, int *pattern, int size) int i; bool ir_avail; + if (!pattern) + return DEVICE_ERROR_INVALID_PARAMETER; + if (size <= 0) { + _E("IR pattern size is invalid"); + return DEVICE_ERROR_INVALID_PARAMETER; + } + ret = device_ir_is_available(&ir_avail); if (!ir_avail) { if (ret < 0) { @@ -76,13 +83,6 @@ int device_ir_transmit(int carrier_frequency, int *pattern, int size) return DEVICE_ERROR_OPERATION_FAILED; } - if (!pattern) - return DEVICE_ERROR_INVALID_PARAMETER; - if (size <= 0) { - _E("IR pattern size is invalid"); - return DEVICE_ERROR_INVALID_PARAMETER; - } - freq_pattern[0] = carrier_frequency; for (i = 1; i <= size; i++) freq_pattern[i] = pattern[i-1]; -- 2.7.4