commit 6b43739e17c16aec0c987bf02302db3b0e4d4aa8
author Wind Yuan <feng.yuan@intel.com> Tue Feb 02 00:31:30 2016 +0800
committer Wind Yuan <feng.yuan@intel.com> Tue Feb 02 00:31:46 2016 +0800
tree c688a759e68127cf030beef58efe26e240987772
parent db3def7daa7a9f68116fb9907561a69f5d7c374b

bugs: fix klockwork scan issues

Signed-off-by: Wind Yuan <feng.yuan@intel.com>

tests/test-device-manager.cpp

diff --git a/tests/test-device-manager.cpp b/tests/test-device-manager.cpp
index 32925f3..ac153f9 100644
--- a/tests/test-device-manager.cpp
+++ b/tests/test-device-manager.cpp
@@ -410,6 +410,7 @@
     while ((opt = getopt_long(argc, argv, short_opts, long_opts, NULL)) != -1) {
         switch (opt) {
         case 'a': {
+            XCAM_ASSERT (optarg);
             if (!strcmp (optarg, "dynamic"))
                 analyzer_type = AnalyzerTypeDynamic;
             else if (!strcmp (optarg, "simple"))
@@ -428,6 +429,7 @@
         }
 
         case 'm': {
+            XCAM_ASSERT (optarg);
             if (!strcmp (optarg, "dma"))
                 v4l2_mem_type = V4L2_MEMORY_DMABUF;
             else if (!strcmp (optarg, "mmap"))
@@ -456,6 +458,7 @@
                                         (unsigned)optarg[3]);
             break;
         case 'd':
+            XCAM_ASSERT (optarg);
             if (!strcmp (optarg, "still"))
                 capture_mode = V4L2_CAPTURE_MODE_STILL;
             else if (!strcmp (optarg, "video"))
@@ -481,9 +484,11 @@
             XCAM_LOG_DEBUG("using USB camera plugged in at node: %s", usb_device_name);
             break;
         case 'R':
+            XCAM_ASSERT (optarg);
             sscanf (optarg, "%d%*c%d", &frame_width, &frame_height);
             break;
         case 'e': {
+            XCAM_ASSERT (optarg);
             if (!strcmp (optarg, "primary"))
                 display_mode = DRM_DISPLAY_MODE_PRIMARY;
             else if (!strcmp (optarg, "overlay"))
@@ -502,6 +507,7 @@
             break;
 #if HAVE_LIBCL
         case 'H': {
+            XCAM_ASSERT (optarg);
             if (!strcasecmp (optarg, "rgb"))
                 hdr_type = CL_HDR_TYPE_RGB;
             else if (!strcasecmp (optarg, "lab"))
@@ -534,6 +540,7 @@
             break;
         }
         case 'T': {
+            XCAM_ASSERT (optarg);
             if (!strcasecmp (optarg, "yuv"))
                 tnr_type = CL_TNR_TYPE_YUV;
             else if (!strcasecmp (optarg, "rgb"))
@@ -547,6 +554,7 @@
             break;
         }
         case 'L': {
+            XCAM_ASSERT (optarg);
             if (atoi(optarg) < 0 || atoi(optarg) > 255) {
                 print_help (bin_name);
                 return -1;
@@ -563,6 +571,7 @@
             break;
         }
         case 'P': {
+            XCAM_ASSERT (optarg);
             if (!strcasecmp (optarg, "basic"))
                 pipeline_mode = CL3aImageProcessor::BasicPipelineProfile;
             else if (!strcasecmp (optarg, "advance"))
@@ -576,6 +585,7 @@
             break;
         }
         case 'C': {
+            XCAM_ASSERT (optarg);
             if (!strcmp (optarg, "bayer"))
                 capture_stage = CL3aImageProcessor::BasicbayerStage;
             break;