From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bcmv-tmail01.ecl.ntt.co.jp (bcmv-tmail01.ecl.ntt.co.jp [124.146.185.148]) by dpdk.org (Postfix) with ESMTP id AB1C81B466 for ; Tue, 9 Oct 2018 12:52:15 +0200 (CEST) Received: from bcmv-ns01.ecl.ntt.co.jp (bcmv-ns01.ecl.ntt.co.jp [129.60.83.123]) by bcmv-tmail01.ecl.ntt.co.jp (8.14.4/8.14.4) with ESMTP id w99AqEIa017814; Tue, 9 Oct 2018 19:52:14 +0900 Received: from bcmv-ns01.ecl.ntt.co.jp (localhost [127.0.0.1]) by bcmv-ns01.ecl.ntt.co.jp (Postfix) with ESMTP id 72B3E125; Tue, 9 Oct 2018 19:52:14 +0900 (JST) Received: from localhost.localdomain (unknown [129.60.13.51]) by bcmv-ns01.ecl.ntt.co.jp (Postfix) with ESMTP id 5BF0D119; Tue, 9 Oct 2018 19:52:14 +0900 (JST) From: ogawa.yasufumi@lab.ntt.co.jp To: spp@dpdk.org, ferruh.yigit@intel.com, ogawa.yasufumi@lab.ntt.co.jp Date: Tue, 9 Oct 2018 19:52:02 +0900 Message-Id: <20181009105207.42636-1-ogawa.yasufumi@lab.ntt.co.jp> X-Mailer: git-send-email 2.13.1 X-TM-AS-MML: disable Subject: [spp] [PATCH 0/5] Add error handling for add and del commands X-BeenThere: spp@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Soft Patch Panel List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Oct 2018 10:52:16 -0000 From: Yasufumi Ogawa There is no strict checking for returned value of parsing resource UID for add and del commands. It might cause a fatal error if no existing resource is accessed for inappropriate resource UID. To avoid inappropriate accessing, add checking for the resource UID and correct determining of the returned value. This update also includes refactors of log messages in add_ring_pmd() to create a ring port correctly. Yasufumi Ogawa (5): shared: add error handling for invalid res UID spp_nfv: add error handling for add and del cmd spp_vm: add error handling for add and del cmd spp_nfv: refactor add_ring_pmd spp_vm: refactor add_ring_pmd src/nfv/nfv.c | 66 ++++++++++++++++++++++++++++++++++++----------------- src/shared/common.c | 7 ++++++ src/vm/main.c | 57 +++++++++++++++++++++++++++++++-------------- 3 files changed, 92 insertions(+), 38 deletions(-) -- 2.7.4