Message ID | 1458297963-19897-1-git-send-email-prabhakar.kushwaha@nxp.com |
---|---|
State | Accepted |
Delegated to: | York Sun |
Headers | show
Return-Path: <u-boot-bounces@lists.denx.de> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 3qRMhP5kBtz9sCZ for <incoming@patchwork.ozlabs.org>; Fri, 18 Mar 2016 22:00:37 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 002C2A76BA; Fri, 18 Mar 2016 12:00:36 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Iv05MdmWEYCY; Fri, 18 Mar 2016 12:00:35 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 31566A767D; Fri, 18 Mar 2016 12:00:35 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 7105BA767D for <u-boot@lists.denx.de>; Fri, 18 Mar 2016 12:00:29 +0100 (CET) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id b3qpJF6gjK27 for <u-boot@lists.denx.de>; Fri, 18 Mar 2016 12:00:29 +0100 (CET) X-Greylist: delayed 921 seconds by postgrey-1.34 at theia; Fri, 18 Mar 2016 12:00:24 CET X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0069.outbound.protection.outlook.com [207.46.100.69]) by theia.denx.de (Postfix) with ESMTPS id 8EEF9A762A for <u-boot@lists.denx.de>; Fri, 18 Mar 2016 12:00:24 +0100 (CET) Received: from BY2PR03CA064.namprd03.prod.outlook.com (10.141.249.37) by BL2PR03MB369.namprd03.prod.outlook.com (10.141.89.12) with Microsoft SMTP Server (TLS) id 15.1.434.16; Fri, 18 Mar 2016 10:46:12 +0000 Received: from BY2FFO11FD034.protection.gbl (2a01:111:f400:7c0c::139) by BY2PR03CA064.outlook.office365.com (2a01:111:e400:2c5d::37) with Microsoft SMTP Server (TLS) id 15.1.434.16 via Frontend Transport; Fri, 18 Mar 2016 10:46:11 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none;nxp.com; dmarc=none action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BY2FFO11FD034.mail.protection.outlook.com (10.1.14.219) with Microsoft SMTP Server (TLS) id 15.1.434.11 via Frontend Transport; Fri, 18 Mar 2016 10:46:10 +0000 Received: from b32579-VirtualBox.ap.freescale.net (B32579-02.ap.freescale.net [10.232.40.8]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id u2IAk4Je017582; Fri, 18 Mar 2016 03:46:08 -0700 From: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> To: <u-boot@lists.denx.de> Date: Fri, 18 Mar 2016 16:16:03 +0530 Message-ID: <1458297963-19897-1-git-send-email-prabhakar.kushwaha@nxp.com> X-Mailer: git-send-email 1.9.1 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131027715707705575; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(2980300002)(1109001)(1110001)(339900001)(189002)(199003)(110136002)(104016004)(4326007)(189998001)(50986999)(92566002)(6806005)(77096005)(5008740100001)(2906002)(47776003)(48376002)(106466001)(81166005)(105606002)(5003940100001)(85426001)(229853001)(87936001)(2351001)(86362001)(36756003)(50226001)(50466002)(586003)(1220700001)(1096002)(33646002)(19580395003)(19580405001)(7059030); DIR:OUT; SFP:1101; SCL:1; SRVR:BL2PR03MB369; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; MLV:sfv; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD034; 1:hg0L8JEPqap3sltIIvl7ZDaZz1DZ+ZgvXcVGRypvhPwgtlVvIO+KlXSnmn1l47B9o+Vd2VRyk89zoAh5LK6OileG69eTcyrPdNVfG+dgj3ZzBp5UMAlz5v+4yGjyCAneb8IbeJclNJwmnkAmT/zNsCfCiL26SsfaY2c98nKlIq1hhNK7xbOa5wcGersqDExQIn+/GT4GdbFlB+97Q8TCvdOOS3CjnejohoZ9eYO1ZErTJKACKeHr2S4rfSZZURz4745+sGlBR6G3MiqxkWJaQJot3eZE/C7WJHB3CK8cnfTkZyKHOLEHIlWle29Ce2OCIPfJFGSGk4aK/MbqIUHRQHYFRzqkxqyxHjloikDHqxTzFjxkHqhxuRl64AnS5ws7HuKRqcozJO4t8IH2ctnnl3l4wXjFmtzToyBYbax/EYdEeX12uElvimudMyXPZsE9SluISbGooZH6ME+4jTACEQ7zKg2qH+FF9trIKsKd4JzS0JE8mOCAO9wsIO0XTsjWeTpqLz+qxK9xfmxYNbd3prcv64Jy/L9lQmPeTjtCajiUOutkiLU0G9MbM1qUnRc993GVBz6exKvPtbgZR9CaNftUX9c73N6WykkGWnNbAiq+sHIGxdumtbdPbl7nSaRw MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: f3c6f284-6a0c-4f0b-6f40-08d34f1a84e3 X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB369; 2:UO6Z42u1Oe3a6g6+kgzEoNnodCAO2xO0+mYh+FiTJRNMvZ6JtpVhBFmrT+wVB2iLkGOdyvSUwob0Qr7s8mZHomje3IBCSoU6WFjm2FW89zB0T3G0uCh+G1HQ+i3w005V+wSxPwpcS54vk6hh7basTEX43epXbIUKNRcy98kw8NFU/2Q+Locvn0I+3I0tF3Mz; 3:dli4WV7rN5rRVsuFDbBFsd4uVVLdAbN7X2MCyg+MUUJKhQg18kQlGJVh1HUe3r2tZ/i5ZcN8e+MjFB7EoHhdMH4VY9peXhzbcCuJ1fRDN+aato460UI6dbpVDRTyEKVSFDZcLews8v+AqnZ2UngOXyUpXf6QNk22fF5/d9M24c0XlpFvMgvNNBs6QHI+ldW74j5kt/j9SJuSs5p3BIRzazhzWC1FMSCwrK4Ubnd9s6Q= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BL2PR03MB369; X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB369; 25:i1ULVWGB49bFEc3FBVPBBOhfo81vUehxQczKldMtK7xhUGakK3LcHowwXlLwalkVQP+SdEjCma5sTh05+9eyR8/zyZphgyInB/CLqM4Fer8K9klVRcAO6vz62d1+y/VuR8II3YYvucH0TBez6mkpkwgOfiEVlef7UZH4CjfleAPoN6L9FLT6VSXxYp8tv21CRcuyuZnHTJwe636pro5TePTqm0dskJ55/f/e7gSb2VIWiBv0YN23BEh3LrvDjcKZeGQDiOKpvuMQm8d1SFtr9kDJQMrZMgVmhi+cCaKUygPQrodw5SD5z4IRekbxb/rorAc+dV7XJyEz0ljOYNepIB6koSyVzhdHFfDY69ArAS5xpAZlxpnkDZNYyhXk8uGrX2YzNwcPW4n7fsk9O8apBE/xrn1bFTUV5pj3A/Pa0MhBAf/OoIHDEhxNhopk1Rdb1OtKGpoj/zuNH7mMNex5ASQlTX2PJWXkqL+6VppMII8ubMIfBzX/bUJrn9JzPv3g2bsBVwS9UQKcpwTue94mndPdxQhT8H0kTgAv9HRoF3E= X-Microsoft-Antispam-PRVS: <BL2PR03MB369F8C063C3DF7433335BD3978C0@BL2PR03MB369.namprd03.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(13017025)(13015025)(5005006)(13023025)(13024025)(13018025)(8121501046)(3002001)(10201501046); SRVR:BL2PR03MB369; BCL:0; PCL:0; RULEID:(400006); SRVR:BL2PR03MB369; X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB369; 4:DmaP4Y9+deeANChEC0VRPCAYDFW84uUA/sdVWJsWPTFH79KYSgzhP/Bh4JpOp3kbcSAU1olb1bTO9kmAqU88qak+TZOjbTnsAdOzcRVpjsOSE+d0j6+QiFtDcOkwLzYjNrdZSUCILfaClA47ajVIDknzYizPYh1cgb3boaEVlcqcMf4xaopXf+EhQ594tVsO7ym5XoC5YDax4uTAva9YrUIdNkRWblo2NhrpgxOD8mdRBQdlou8ZIE4xukTFAlVf7CLanQcbAjlIPrx/21ogkUgHjIc6ngBCN2hgxPTKyWEaFpYHvgOwGaZCeIyeQAZ/zMc6t/E0kD9OJKhag/kaj0nA/TMlmUu8xKXGo1t9Anpa3Sxz+Q9T/RoVPGAqWxFoBwGCmZ0/N5zfxjSObT1gs6VzKPsOrWa4sMi4pX6C6vw3ZW8Qyb0OTq4ocHd3XDMsMgIoXXF9p+HAJ3o0lYj2qA== X-Forefront-PRVS: 088552DE73 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BL2PR03MB369; 23:PVXtjIWPKFjHJ5I8nf+DG64oDsLVXg/XP2Dqkdk/Tn?= =?us-ascii?Q?9aF2968MotUqBN34OCMDQmB8JUG7P86APuN4DNez/GK845rM2fVEATuhwtlO?= =?us-ascii?Q?DR7bQXFEkPzic4gdRLjlNuHgLSMSwOP0ynP2PtJRaNeN6AgqBJNafksUGHfT?= =?us-ascii?Q?L7l0iyLcTNqIA0Rzz3aRwpj2+Y92XKJ8qeszKSDWc3MAFuuzDVfDED8gbgL9?= =?us-ascii?Q?VT976tKaIC/TftAZoizGmej9J5tHMP+j4J84l1Ofhlq9ghwEFmERRq1neAUr?= =?us-ascii?Q?A8Xg+tMLfFXgAWcgTQuTn9j7CUJvQdqPIc0QHKaIiZ8HMQuXwuh20gQeI/Ex?= =?us-ascii?Q?m+w8mkwcsPOLab4wCwN+OMD0yV/JOZK3GzTWL5LPhttUg6xkRp9gDYHEXLRj?= =?us-ascii?Q?DT2fifEqH+9suzkmvllLknfJZiAbJqu8vFOCRqPX1nIK3E07elItLPQyWxXI?= =?us-ascii?Q?MsnSzZD/vkahSkVqwj1ZUYse4rT8LuKKznhPvZc/NWPiJXq9uNBWhbHc5Mj4?= =?us-ascii?Q?YqHChpz7su8zPNl+yQW1IRaGQFGdBDuPxLOunKlvuZvkPYssiQQiXQcnU8PK?= =?us-ascii?Q?agh+3PaUMxYJG8/2u3vG3NTd1rQafnykNxXIjbvc5OiKaa5snpq5ghP6gygE?= =?us-ascii?Q?aKVDFCA5H1iXdeslJtbduJs1kOKpq2hwX5mtksE+1j5w6gLSwVI8YkUxKuP2?= =?us-ascii?Q?fWWSzZCZWUDbZxbiP+5R4NZIOq8WwGJKE0p1t5KzUX9mtmrRNiqHv30d/LYG?= =?us-ascii?Q?PwzuKW7ovnzOjkxrwITKVqqDSQ55QHaLihQZhQgoAujUPJNffknfyiSveOXd?= =?us-ascii?Q?1vdza542Pbj7WgNJgvdwSCm19FTjvly/ZYVrw5U9hHavTH39eG7F0R1oz19g?= =?us-ascii?Q?rfynFjTU0FddEUHXMRpUem5EECBU1HtszOdONU1ywkP4RKPz7DcVbfEZoZhc?= =?us-ascii?Q?k4Oh4pgC+YRe4GCMx1TpPFSHMjan1xHzGutBVPuaHMKvXOAF1kYVPfN62VBQ?= =?us-ascii?Q?4=3D?= X-Microsoft-Exchange-Diagnostics: 1; BL2PR03MB369; 5:mgo6gzzVLvNXtFbE6swUuZriygCRrQJvclpf11NKh8nXdIPPDwP9Xq9JLUGJlTWghBjDNiq3Uf2VJ+WgpkgLtBFdHOehmoxyrF8/35nKkf0kia5GXWr/A95yr6okLdTrft7WWZPmFamtAQea7tJF83qDS2FMli2DSAD5hJvm5Z0=; 24:IGQ7vUJlyHki/MILq1V3vtD64jDFhmnoP4EunhvkPGa3/vLazxMqtS2u1W7n1Ab64aksduNgUy1Xrq2y57rHfeiLybSpVzPONuurUNKXpRw= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Mar 2016 10:46:10.4585 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR03MB369 Subject: [U-Boot] [PATCH] driver: net: fsl-mc: Check NULL before pointer dereference X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list List-Id: U-Boot discussion <u-boot.lists.denx.de> List-Unsubscribe: <http://lists.denx.de/mailman/options/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=unsubscribe> List-Archive: <http://lists.denx.de/pipermail/u-boot/> List-Post: <mailto:u-boot@lists.denx.de> List-Help: <mailto:u-boot-request@lists.denx.de?subject=help> List-Subscribe: <http://lists.denx.de/mailman/listinfo/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=subscribe> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de> |
On 03/18/2016 03:46 AM, Prabhakar Kushwaha wrote: > NULL pointer should be checked before any dereference. This patch > move memest after the NULL pointer check. > > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> > Reported-by: Jose Rivera <german.rivera@nxp.com> > --- > drivers/net/fsl-mc/dpio/qbman_sys.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied to u-boot-fsl-qoriq master. Awaiting upstream. Thanks. York
diff --git a/drivers/net/fsl-mc/dpio/qbman_sys.h b/drivers/net/fsl-mc/dpio/qbman_sys.h index 235d641..7a537fb 100644 --- a/drivers/net/fsl-mc/dpio/qbman_sys.h +++ b/drivers/net/fsl-mc/dpio/qbman_sys.h @@ -255,11 +255,11 @@ static inline int qbman_swp_sys_init(struct qbman_swp_sys *s, s->addr_cena = d->cena_bar; s->addr_cinh = d->cinh_bar; s->cena = (void *)valloc(CONFIG_SYS_PAGE_SIZE); - memset((void *)s->cena, 0x00, CONFIG_SYS_PAGE_SIZE); if (!s->cena) { printf("Could not allocate page for cena shadow\n"); return -1; } + memset((void *)s->cena, 0x00, CONFIG_SYS_PAGE_SIZE); #ifdef QBMAN_CHECKING /* We should never be asked to initialise for a portal that isn't in
NULL pointer should be checked before any dereference. This patch move memest after the NULL pointer check. Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> Reported-by: Jose Rivera <german.rivera@nxp.com> --- drivers/net/fsl-mc/dpio/qbman_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)