From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5C724A04B4; Wed, 26 Aug 2020 14:09:32 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C83FB58C4; Wed, 26 Aug 2020 14:09:31 +0200 (CEST) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-eopbgr60076.outbound.protection.outlook.com [40.107.6.76]) by dpdk.org (Postfix) with ESMTP id 095432AB; Wed, 26 Aug 2020 14:09:31 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=C3D8Ywo4LyPKDYq7SEUZ/cZTYs2TXc/qXNqRYbflOcdpU1yFWLsj1LahNdQZgDqEBFfU3OOxZUwvmNXHz2ieoxQh2o7iXleNnZcSJMMQQXuOGRCJBPMyUBL+99dFjkwCv1vg/qEaJkNBO3jdkzPLc3N1cOTaThuxJEUD1tUz1xH1M3suIwbCmiHe7hRMWs3fk2ltkzUvyi3Wc0F8LpbdBd43vsGi5EWDep25HOc1EjpwX8OvqVI0teGdhEs7TcYK4pvIk6CvMgxi/lMXWlXefQ6XHd/lPtaJF4BnAwYC3ahapMFRCrI+anFqvfJLNLM0q85yE8uEBpuk5BeY302vXw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5seg+omSRlQ9pjQeruUcp01W7Si/g1Yu/FRunHpAURU=; b=FD+sJFiYLV7HP4X6IVritMcu5xPrsywgBT/P5KsEOM/7XpxKj0az8Nh9nbUwGgbRh2jmf8YocFLad3OlyPf1Mmm4iP/B+Go+BGyKhg6WOKRJUSIRarxINF6oHzSkwowW5IODiw4itbr9T2kN4j+tVHTYj89fOX9KRAwd15ufTsauwYkx3cSj1NEDo+0o0vFR+aBXvvRstyLd3bvC1hfzopAdN+p8IM8s14zD+ORWQh9ePmxKPfX7YBHB+NTjafyKiBE22Eja9jv17ZlFyF4axcc4j5V68EaZQKZGtKfN97P/EVKys36axpFZRB5HeY5eD0fRQVad6mGREJYah6+Wrg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector2-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5seg+omSRlQ9pjQeruUcp01W7Si/g1Yu/FRunHpAURU=; b=FKrY+QmILt8OqjQS87uvv1pfZoJ2tfkCPzD1a2ppaCD0YR+bbM3FSLMwCC5xYrVvYRhWxmc6ZglCZRjU7phRyjBwZVtiXOIRxBSFy1iL00xOVYngUA3EvtSQkwzJqqUM51QESg8YtNtjgbLAQiCNyB89R7AByUR6tIBdJxHS/cE= Authentication-Results: dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=none action=none header.from=oss.nxp.com; Received: from VI1PR04MB5837.eurprd04.prod.outlook.com (2603:10a6:803:ec::28) by VI1PR0402MB2719.eurprd04.prod.outlook.com (2603:10a6:800:b0::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3305.26; Wed, 26 Aug 2020 12:09:29 +0000 Received: from VI1PR04MB5837.eurprd04.prod.outlook.com ([fe80::c506:dc43:37b2:7d52]) by VI1PR04MB5837.eurprd04.prod.outlook.com ([fe80::c506:dc43:37b2:7d52%6]) with mapi id 15.20.3305.031; Wed, 26 Aug 2020 12:09:29 +0000 To: wangyunjian , dev@dpdk.org, hemant.agrawal@nxp.com, sachin.saxena@nxp.com Cc: jerry.lilijun@huawei.com, xudingke@huawei.com, stable@dpdk.org References: From: "Sachin Saxena (OSS)" Message-ID: <93b330ef-5649-0583-5cd2-cd9ab2c9dcae@oss.nxp.com> Date: Wed, 26 Aug 2020 17:38:53 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-ClientProxiedBy: SG2PR03CA0133.apcprd03.prod.outlook.com (2603:1096:4:c8::6) To VI1PR04MB5837.eurprd04.prod.outlook.com (2603:10a6:803:ec::28) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [192.168.1.10] (106.215.58.204) by SG2PR03CA0133.apcprd03.prod.outlook.com (2603:1096:4:c8::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3348.5 via Frontend Transport; Wed, 26 Aug 2020 12:09:26 +0000 X-Originating-IP: [106.215.58.204] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 9dcf8f12-2796-4a21-60b9-08d849b8e1e8 X-MS-TrafficTypeDiagnostic: VI1PR0402MB2719: X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7219; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: kzK7hvNOxA0lut8AOO0oHXlFGKCJ9jm8OzhBfOS/mCO1mjfXtqtvZFEEdmSqYqiaYGsww/G5bSNVqTEBlcybGXR6Aa0DRe4HW6/Ka+7OuGBiO0iwtL3nxS1Olb2L5AmhUaabJzfTZzXenuxPqrXwYu2o5J3Qzx43dlBRZ6vbjxK9PryOhNpq7H0orfL42mmZv0wnq1tewqWKC+MdzLZU9kmQ0B3noSkj2nAVX17Bm6xoebtYzPxsRtSVs11Qd719l1jYQnhPy5rQ542DeUp1nSnb3SnoETM2SDacP01P7lCGW6OBcG+MxAmeyi2fmYIOElLalDO6moCxVyzY2xeDdNyQPKVSGe+unMmL1E9NomrMXpvmVli6E3IF2o9YV1ZG X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR04MB5837.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(136003)(376002)(346002)(39860400002)(366004)(396003)(83380400001)(66946007)(6486002)(478600001)(5660300002)(31686004)(4326008)(1006002)(6666004)(66556008)(66476007)(8936002)(2906002)(16526019)(53546011)(316002)(26005)(86362001)(16576012)(8676002)(52116002)(186003)(2616005)(956004)(31696002)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData: LtWt9NTLMyTgQPtm4+M4fcE3MGHFOFy1+hFzBAEQq0TV1k1gMpOgu9UEfRkyioEzrUIYYLHVb/r9ddKYaFn53ZykwQCGI4+icEXin58zZMye6V8AvrUcFYZgkCzV8SoZ1CLUKVu9f7O0zvwdIMZcaVCqdvO7hiyRac2sBP7KTwvDkfTu7ECJCtGUQOvlhWQ5VHyXNfORZBsYnVgZjuQSDOpgluV7J+L+IpmoUvwVGckAKZErPuHGvW5juqBmRRDAlxvwjnj/zFyVR1H4kHMIcb51rKyceX/QBCpaeJ0/aeTUHY8ytQYpCS8HgvrkL5aYZOqjv8P6SKXp8I/goKc1+MHAakSBqE0GGxYQ6+ZKaVjq3h5BoaZ28UlDl4NPJYINZNXFXoWt3FGJqpUvxksfQHoyF7C5GjDIfkutXH+Ruj4FQpLXTeXbG/9zYxIJadWePEyk49haMp20BNlJ5RlCzSlOK7Lk+AIKwT48mh0e+R9BJl81anjG3EwIiN0GxlksoIBvsOKNeUffYSClxOZRp7YJjkgyKzW2/rymgnDFK8QB5qWnDggFqKQIObtnVsdUGl225slpOpXP7gU6r+iHnnLBu9k3tWBrsSmVQyBdZ0ojhUq7dnB3FApW8N9zoPmjDcO+MGbynhPu0L/ptkTOhA== X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9dcf8f12-2796-4a21-60b9-08d849b8e1e8 X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5837.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Aug 2020 12:09:28.9455 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 4SGZLrrd0/OjskAyoGxqp2m21pE7hWO+D+9Xa8bxSxWKlIx/78tJcYeSJLt8xb1DtALFffRJd7ZYvuNgtsQNsw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0402MB2719 Subject: Re: [dpdk-dev] [PATCH] bus/dpaa: fix fd check before close X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Thanks Yunjian for the fix. Acked-by: Sachin Saxena On 26-Aug-20 5:24 PM, wangyunjian wrote: > From: Yunjian Wang > > The fd is possibly a negative value while it is passed as an > argument to function "close". Fix the check to the fd. > > Fixes: b9c94167904f ("bus/dpaa: decouple FQ portal alloc and init") > Cc: stable@dpdk.org > > Signed-off-by: Yunjian Wang > --- > drivers/bus/dpaa/base/qbman/qman_driver.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/bus/dpaa/base/qbman/qman_driver.c b/drivers/bus/dpaa/base/qbman/qman_driver.c > index 1166d68e2..1bff0bc2f 100644 > --- a/drivers/bus/dpaa/base/qbman/qman_driver.c > +++ b/drivers/bus/dpaa/base/qbman/qman_driver.c > @@ -142,7 +142,7 @@ struct qman_portal *fsl_qman_fq_portal_create(int *fd) > struct qm_portal_config *q_pcfg; > struct dpaa_ioctl_irq_map irq_map; > struct dpaa_ioctl_portal_map q_map = {0}; > - int q_fd = 0, ret; > + int q_fd = -1, ret; > > q_pcfg = kzalloc((sizeof(struct qm_portal_config)), 0); > if (!q_pcfg) { > @@ -191,7 +191,7 @@ struct qman_portal *fsl_qman_fq_portal_create(int *fd) > err: > if (portal) > qman_free_global_portal(portal); > - if (q_fd) > + if (q_fd != -1) > close(q_fd); > process_portal_unmap(&q_map.addr); > kfree(q_pcfg);