test mbuf attach
[dpdk.git] / app / test / test_cmdline_num.c
index 0e416ae..a6ad955 100644 (file)
@@ -1,37 +1,8 @@
-/*-
- *   BSD LICENSE
- * 
- *   Copyright(c) 2010-2013 Intel Corporation. All rights reserved.
- *   All rights reserved.
- * 
- *   Redistribution and use in source and binary forms, with or without 
- *   modification, are permitted provided that the following conditions 
- *   are met:
- * 
- *     * Redistributions of source code must retain the above copyright 
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright 
- *       notice, this list of conditions and the following disclaimer in 
- *       the documentation and/or other materials provided with the 
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its 
- *       contributors may be used to endorse or promote products derived 
- *       from this software without specific prior written permission.
- * 
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- * 
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2010-2014 Intel Corporation
  */
 
+#include <stdio.h>
 #include <string.h>
 #include <inttypes.h>
 
@@ -245,19 +216,6 @@ const char * num_invalid_strs[] = {
                "\0",
 };
 
-#define NUM_POSITIVE_STRS_SIZE \
-       (sizeof(num_valid_positive_strs) / sizeof(num_valid_positive_strs[0]))
-#define NUM_NEGATIVE_STRS_SIZE \
-       (sizeof(num_valid_negative_strs) / sizeof(num_valid_negative_strs[0]))
-#define NUM_POSITIVE_GARBAGE_STRS_SIZE \
-       (sizeof(num_garbage_positive_strs) / sizeof(num_garbage_positive_strs[0]))
-#define NUM_NEGATIVE_GARBAGE_STRS_SIZE \
-       (sizeof(num_garbage_negative_strs) / sizeof(num_garbage_negative_strs[0]))
-#define NUM_INVALID_STRS_SIZE \
-       (sizeof(num_invalid_strs) / sizeof(num_invalid_strs[0]))
-
-
-
 static int
 can_parse_unsigned(uint64_t expected_result, enum cmdline_numtype type)
 {
@@ -315,6 +273,7 @@ can_parse_signed(int64_t expected_result, enum cmdline_numtype type)
        case UINT64:
                if (expected_result < 0)
                        return 0;
+               break;
        case INT8:
                if (expected_result > INT8_MAX || expected_result < INT8_MIN)
                        return 0;
@@ -346,18 +305,17 @@ test_parse_num_invalid_param(void)
        token.num_data.type = UINT32;
 
        /* copy string to buffer */
-       rte_snprintf(buf, sizeof(buf), "%s",
-                       num_valid_positive_strs[0].str);
+       strlcpy(buf, num_valid_positive_strs[0].str, sizeof(buf));
 
        /* try all null */
-       ret = cmdline_parse_num(NULL, NULL, NULL);
+       ret = cmdline_parse_num(NULL, NULL, NULL, 0);
        if (ret != -1) {
                printf("Error: parser accepted null parameters!\n");
                return -1;
        }
 
        /* try null token */
-       ret = cmdline_parse_num(NULL, buf, (void*)&result);
+       ret = cmdline_parse_num(NULL, buf, (void*)&result, sizeof(result));
        if (ret != -1) {
                printf("Error: parser accepted null token!\n");
                return -1;
@@ -365,14 +323,15 @@ test_parse_num_invalid_param(void)
 
        /* try null buf */
        ret = cmdline_parse_num((cmdline_parse_token_hdr_t*)&token, NULL,
-                       (void*)&result);
+               (void*)&result, sizeof(result));
        if (ret != -1) {
                printf("Error: parser accepted null string!\n");
                return -1;
        }
 
        /* try null result */
-       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*)&token, buf, NULL);
+       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*)&token, buf,
+               NULL, 0);
        if (ret == -1) {
                printf("Error: parser rejected null result!\n");
                return -1;
@@ -395,13 +354,6 @@ test_parse_num_invalid_param(void)
                return -1;
        }
 
-       /* try null buf */
-       ret = cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token, NULL, sizeof(buf));
-       if (ret != -1) {
-               printf("Error: help function accepted null buffer!\n");
-               return -1;
-       }
-
        /* coverage! */
        ret = cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token, buf, sizeof(buf));
        if (ret < 0) {
@@ -427,13 +379,13 @@ test_parse_num_invalid_data(void)
                token.num_data.type = type;
 
                /* test full strings */
-               for (i = 0; i < NUM_INVALID_STRS_SIZE; i++) {
+               for (i = 0; i < RTE_DIM(num_invalid_strs); i++) {
 
                        memset(&result, 0, sizeof(uint64_t));
                        memset(&buf, 0, sizeof(buf));
 
                        ret = cmdline_parse_num((cmdline_parse_token_hdr_t*)&token,
-                                       num_invalid_strs[i], (void*)&result);
+                               num_invalid_strs[i], (void*)&result, sizeof(result));
                        if (ret != -1) {
                                /* get some info about what we are trying to parse */
                                cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token,
@@ -466,15 +418,16 @@ test_parse_num_valid(void)
                token.num_data.type = type;
 
                /* test positive strings */
-               for (i = 0; i < NUM_POSITIVE_STRS_SIZE; i++) {
+               for (i = 0; i < RTE_DIM(num_valid_positive_strs); i++) {
                        result = 0;
                        memset(&buf, 0, sizeof(buf));
 
                        cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token,
                                        buf, sizeof(buf));
 
-                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token, num_valid_positive_strs[i].str,
-                                       (void*)&result);
+                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token,
+                               num_valid_positive_strs[i].str,
+                               (void*)&result, sizeof(result));
 
                        /* if it should have passed but didn't, or if it should have failed but didn't */
                        if ((ret < 0) == (can_parse_unsigned(num_valid_positive_strs[i].result, type) > 0)) {
@@ -493,15 +446,16 @@ test_parse_num_valid(void)
                }
 
                /* test negative strings */
-               for (i = 0; i < NUM_NEGATIVE_STRS_SIZE; i++) {
+               for (i = 0; i < RTE_DIM(num_valid_negative_strs); i++) {
                        result = 0;
                        memset(&buf, 0, sizeof(buf));
 
                        cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token,
                                        buf, sizeof(buf));
 
-                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token, num_valid_negative_strs[i].str,
-                                       (void*)&result);
+                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token,
+                               num_valid_negative_strs[i].str,
+                               (void*)&result, sizeof(result));
 
                        /* if it should have passed but didn't, or if it should have failed but didn't */
                        if ((ret < 0) == (can_parse_signed(num_valid_negative_strs[i].result, type) > 0)) {
@@ -542,15 +496,16 @@ test_parse_num_valid(void)
                token.num_data.type = type;
 
                /* test positive garbage strings */
-               for (i = 0; i < NUM_POSITIVE_GARBAGE_STRS_SIZE; i++) {
+               for (i = 0; i < RTE_DIM(num_garbage_positive_strs); i++) {
                        result = 0;
                        memset(&buf, 0, sizeof(buf));
 
                        cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token,
                                        buf, sizeof(buf));
 
-                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token, num_garbage_positive_strs[i].str,
-                                       (void*)&result);
+                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token,
+                               num_garbage_positive_strs[i].str,
+                               (void*)&result, sizeof(result));
 
                        /* if it should have passed but didn't, or if it should have failed but didn't */
                        if ((ret < 0) == (can_parse_unsigned(num_garbage_positive_strs[i].result, type) > 0)) {
@@ -569,15 +524,16 @@ test_parse_num_valid(void)
                }
 
                /* test negative strings */
-               for (i = 0; i < NUM_NEGATIVE_GARBAGE_STRS_SIZE; i++) {
+               for (i = 0; i < RTE_DIM(num_garbage_negative_strs); i++) {
                        result = 0;
                        memset(&buf, 0, sizeof(buf));
 
                        cmdline_get_help_num((cmdline_parse_token_hdr_t*)&token,
                                        buf, sizeof(buf));
 
-                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token, num_garbage_negative_strs[i].str,
-                                       (void*)&result);
+                       ret = cmdline_parse_num((cmdline_parse_token_hdr_t*) &token,
+                               num_garbage_negative_strs[i].str,
+                               (void*)&result, sizeof(result));
 
                        /* if it should have passed but didn't, or if it should have failed but didn't */
                        if ((ret < 0) == (can_parse_signed(num_garbage_negative_strs[i].result, type) > 0)) {