Message ID | 20151005155103.10841.38132.stgit@tlendack-t1.amdoffice.net |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show
Return-Path: <netdev-owner@vger.kernel.org> X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id C476114029C for <patchwork-incoming@ozlabs.org>; Tue, 6 Oct 2015 02:51:17 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751587AbbJEPvN (ORCPT <rfc822;patchwork-incoming@ozlabs.org>); Mon, 5 Oct 2015 11:51:13 -0400 Received: from mail-bn1bon0090.outbound.protection.outlook.com ([157.56.111.90]:45632 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751055AbbJEPvL (ORCPT <rfc822;netdev@vger.kernel.org>); Mon, 5 Oct 2015 11:51:11 -0400 Received: from CY1PR1201CA0009.namprd12.prod.outlook.com (10.169.17.147) by BN4PR12MB0851.namprd12.prod.outlook.com (10.164.59.13) with Microsoft SMTP Server (TLS) id 15.1.286.20; Mon, 5 Oct 2015 15:51:08 +0000 Received: from BY2NAM03FT028.eop-NAM03.prod.protection.outlook.com (2a01:111:f400:7e4a::206) by CY1PR1201CA0009.outlook.office365.com (2a01:111:e400:5b9a::19) with Microsoft SMTP Server (TLS) id 15.1.286.20 via Frontend Transport; Mon, 5 Oct 2015 15:51:08 +0000 Authentication-Results: spf=none (sender IP is 165.204.84.222) smtp.mailfrom=amd.com; davemloft.net; dkim=none (message not signed) header.d=none; davemloft.net; dmarc=permerror action=none header.from=amd.com; Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) Received: from atltwp02.amd.com (165.204.84.222) by BY2NAM03FT028.mail.protection.outlook.com (10.152.84.238) with Microsoft SMTP Server id 15.1.293.9 via Frontend Transport; Mon, 5 Oct 2015 15:51:07 +0000 X-WSS-ID: 0NVR813-08-8RD-02 X-M-MSG: Received: from satlvexedge02.amd.com (satlvexedge02.amd.com [10.177.96.29]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by atltwp02.amd.com (Axway MailGate 5.3.1) with ESMTPS id 27C3BD1603A; Mon, 5 Oct 2015 11:51:02 -0400 (EDT) Received: from SATLEXDAG01.amd.com (10.181.40.3) by SATLVEXEDGE02.amd.com (10.177.96.29) with Microsoft SMTP Server (TLS) id 14.3.195.1; Mon, 5 Oct 2015 10:51:33 -0500 Received: from tlendack-t1.amdoffice.net (10.180.168.240) by SATLEXDAG01.amd.com (10.181.40.3) with Microsoft SMTP Server id 14.3.266.1; Mon, 5 Oct 2015 11:51:04 -0400 Subject: [PATCH] amd-xgbe: Check for successful buffer allocation before use From: Tom Lendacky <thomas.lendacky@amd.com> To: <netdev@vger.kernel.org> CC: David Miller <davem@davemloft.net> Date: Mon, 5 Oct 2015 10:51:03 -0500 Message-ID: <20151005155103.10841.38132.stgit@tlendack-t1.amdoffice.net> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:165.204.84.222; CTRY:US; IPV:NLI; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(2980300002)(428002)(189002)(199003)(5007970100001)(97746001)(5003600100002)(50986999)(4001540100001)(33646002)(83506001)(97736004)(23676002)(229853001)(4001350100001)(50466002)(54356999)(2351001)(19580405001)(105586002)(19580395003)(103116003)(11100500001)(106466001)(5001830100001)(5008740100001)(87936001)(5001860100001)(64706001)(62966003)(68736005)(5001920100001)(53416004)(47776003)(189998001)(86362001)(77096005)(101416001)(77156002)(110136002)(46102003)(92566002)(71626007); DIR:OUT; SFP:1101; SCL:1; SRVR:BN4PR12MB0851; H:atltwp02.amd.com; FPR:; SPF:None; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN4PR12MB0851; 2:e1V6WfYy9i4wLYYIt6iqMIZ9jPEV1tKc8dGCL2Dd+OV9DTkN19V/FDLZ1avaCosp1z2eZzOfvw/7WliGWWiHJhYo/pSLHE4KejlhLhwVmZX9//QTKuuhl8PhkzQeiDnPV891MsjEads1FPOsiv2yKaW+1D5mD2Lb466gnwe7+k0=; 3:pZ3sRdT8UegvUj4HNKKkLVhBzHTXSxd1d4LF6vorBxrfZPvWdkaIDJs8mQZV0VynhZJglrLZu1UYMN6fXQQAZJiIUCrr3QmK5tCjHBYKFI0yzBm3daoJl8855Rzbyc8nw4dR6s4cYiGYwNK2uCWx3cznV2otqiT4XOuPpyRxceMUs4VLNad0RuUqVOPuG6cTOrp6ke96ouNtbXTo4Ldl9nYkGOFzPlnPe5x985YmqXBtlHlAoBwf4MS5QZNIRyOc; 25:HrDOuiySyCTJFxnfN1TOasjpZ09jeQd8FthmPF97Qk36mfs07UyYwadeJ8Xnq8jmlie5U07IBayFzQhRGSBSMBUWiiFwL4Ch90ThzGlFkm6sFGIsjfWkxRfMQ1T8bE/UGiw5aR+FrAkuYCD8o5dtllJcX3g0nzaokqfiiK4/gx8aP/NrZSnBubhu+N3aLhPm9Y6lt8soxzBlUx/3jqMHHqUkSVRrkonKfvESDmgHRB3Ntd9iMtC7InlVJM2HDr7jWoYmsPA9n/2b//5BluTt3g== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN4PR12MB0851; X-Microsoft-Exchange-Diagnostics: 1; BN4PR12MB0851; 20:SPq42/VWicCjHIa/02OlZvCrs33xteH3lAWazVe95BeuAJynrvH78Uj2lpDk7JR7m5jVx61Z8Cn5t+WIIni2gPm8YZUAFm/9LiSPKerW6iuvNjSIqhWQVyzR8vrgnje5ZE6sM1k852Sh2ZlrxrpgjRHVvpmrK92pGjJezWkszQxmM432f55vYbFbPNmunYpXIBYivCxSArM+JUhgjwYlbfUrGWpot2UfbxVE0b0Fk/I4QiJLzaIprx/VX8W14fm3Pj30fYVo/Bt03fxjfF8CYhRcW1cnTc8NpFVp7AlXFKgShIxe9uX85fQShzgNdA3hZjQYlg0HIAuW/DG5e2X98xKa5nWwcwvx7wTuvQMywTDEhToZ790a8z/kZlnlPtal97454PQAia23NTFpG3Z2VYDiIFGHzfeTVqlmgYvO8BeTyxHtUHgXrCJF1jBO7qUXsl1GXs0Wr2bCr9zvjPln/nL4dr8Wmci5Ga7XDnNVsCTx75a0RpAnSjy4JGaPGiYC; 4:hm8rxFusuzSicJamUdDAQdJEqYD4sajAoNv+BbbYZ5zxyQ1IeJT4JnsRroplYDmTyB9UOlsHuabUDSY2ZF3COwZSH5ekTIixQ+X3Bdc0zgq3qR2YPqkgYpcYiyYOFwvjl+8PEePuqEkvVJT7Idx8uX7gmZbUjVMU9pp0YQ+rEy0wXZR8GNZHH/LzS+3kJjjTdxxs5aiBrfpFjCIdG1pffoqGmO6F/LNiH+gBlNsGknjmV0duGdIdQmFNDUUutpkxXYwFJqRDfAl6kKEoJm9yz/kaHbV+/mTgdJBmr6bG17eFtX/EsNWfVFmoGv7O3+pAToe1gyyXOUF5jnIgBC1MFPUT9N7TT9nlXegzSwtJC00= X-Microsoft-Antispam-PRVS: <BN4PR12MB08514B02A2F4352B74BD1025EC480@BN4PR12MB0851.namprd12.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(8121501046)(520078)(5005006)(3002001); SRVR:BN4PR12MB0851; BCL:0; PCL:0; RULEID:; SRVR:BN4PR12MB0851; X-Forefront-PRVS: 07200C0526 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTjRQUjEyTUIwODUxOzIzOkt6eVZjMlVBeVlpajVxT1RGRXpnVE1QcmVG?= =?utf-8?B?Zy9xSi9ERlhES0puRjlYYit4LzhIV2RPRDNtRldhWDBDZCtzZGVKTXlUQ1dk?= =?utf-8?B?NnJVdWNEc210OXViblpnSDY3aENhMmpTQkk4U2tEbU9XTDBRVzArWGNCZEtG?= =?utf-8?B?ZUNDMUtkZVRqVHNMNVBIQkpzOTFMR2J1QW1jUDF3STJtMTRYRDVSUnNqMWhY?= =?utf-8?B?U3UrVDdtQm0xdUplVFRsRk5aMEdNY01ncjNERWtTeEUvOW1iT2tZSkQzZXdY?= =?utf-8?B?c3JoNkRYcTJWMGF0Umx1V2ZHUWNkM1JTZ1F3di9ZWlhWRVFjVGs0T1hCU05t?= =?utf-8?B?UnBBMzNyOWlrZW9qT1lLNTJ0K0hzem9pa3VvWXFlcUFsVElyZHhzU3d6NDha?= =?utf-8?B?TlQxS0s3eTkzT1ZNUnZHbG9yTFVPUllVb29VcGltVU9tTEUwSlFHN0IxWVBi?= =?utf-8?B?YXZwTGN2TDB2WGkwUktjbWV3R0Q4RW5RSXdJdGwwd2VDdFBYZjcwVnlFTjhx?= =?utf-8?B?alhGWW12NktTUTdOZlJZUnBwZ3U2TlJZdUZHZ0pEcGt0eVdEK1ZiSjFMcnI2?= =?utf-8?B?M1FmNUdWRG16enM5ZE5kNk1ibllHUDM2Umh5SHJzWExuMk5WRDNaOWFpSHI0?= =?utf-8?B?Vm5yTDdtNEV0OXNzeHM1MUNWaEY4VTJVR2tFMUxLaHlKZEsvRHNxUjRuWjJH?= =?utf-8?B?cFo1Y0V4azJ1QmdybURNMGlSQzg2N2FQeUlZcEVZRVM0RzUwNkFaaVozVFgw?= =?utf-8?B?NkxsYk1sRTlJZDBJUXd5LzJZS08zdTJmUy9lTVByZTUwSDcvSFFYZUU1T3lY?= =?utf-8?B?czRCTnlqVkR5MHlWQldITm5tQU5zeit5cW96ZnpyUVpxTTJER2RQNHdYMHNK?= =?utf-8?B?N2o0czFUdTVVVDAwWnJJdWVrU3lEN3diNEwwZEZKalFkQU9nN2xReWhacFBz?= =?utf-8?B?WlZXbjZ5b2FtQU5rejh1cXdDR3ZyV2Zla2ZwRGxtV2RISjVnVkJnLzQ2Zkw5?= =?utf-8?B?WnNlTDNJVW9aZm9MSWFlU3dJTHQ2S1BRZVBTV1JoVXNlK1NFMlY4OHlsS3VJ?= =?utf-8?B?L0JPbFNlR3NtcmFIVzJWZklBMFpVK3MzaDB2TFl6OXEvZGcvcVFFZ1NGUDNK?= =?utf-8?B?cGtvSEk1WE1XSHhBVm9DcUFRVlQ4OXFuRHBPRXlUNGt6cE14clNkaU1BZy9V?= =?utf-8?B?RnVmRmNGeURXTlVxNWV4VnlxVjRFTWFGMFYvd2s2RVdudlN2cGNsd3MrdUtw?= =?utf-8?B?dUJwYkVaQ0R3S2RzUGczWXZvNDFmL2VtOVJNWXFUcVFqUEVyL2RyWnZ5aVVq?= =?utf-8?B?RUtUZEhzV3UvQzNTRWZHUithWDNnUk1zQ2Nta1hhZXB2ZnNCNlB4YVh1NHdP?= =?utf-8?B?TkhQeHhCM3NGNmRVRnVYMXNtWEppRHRGdy9MTXpocGRCZGNqa3ljYkdmWnBh?= =?utf-8?B?NHNHOVpwcDlQWDZ5aW5IU3hLNVF1dFViaGxid2dRbzQwb3hNSWJ5NStqVUtu?= =?utf-8?B?cmNRMXJ4TEFpaU1WN0NxZjFqVGpaRHZJL3dvYWt6eGJ3Q0RqQ013Ylp3Vlow?= =?utf-8?B?dmRHWXowRThtN2lGb0d3NnM1Q0xmSFFwcEtGVWlVZXduR2x4RDFMSStxaE10?= =?utf-8?Q?SnBQtc4WbAUnAQs5b0RU?= X-Microsoft-Exchange-Diagnostics: 1; BN4PR12MB0851; 5:Qw3f0bz+62aIo9+NmkcFoW5PeBVOtoWDHIlIyblnOP/VnSqbz1b0/fQJkQT2tZTrUcDaviKVZKkyXodyzPaCE0QtuOCOvCEkwoA4JZSVv+X3hw7CSHqAD4+dXHefxeU77u0Bwp5MRx7st4BLVkDMew==; 24:snHK+m5S6MFLer1Loooe4tZBvQkYiT2FTaVShDKPc2QOqFcKtMB3Coy2MKgl6OW5xJSkB4lf524hlXhzRtNJY8QfdqyRcQnHatcyDLV9Q68=; 20:Ay7Y5no5Vq03X+66pC7Kdaerc0TubtlgWCAAsDvizF2ujn8y1OH6IJkuLfolOWh9w7JfvSbwRd6eT79YoB+0nQ== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Oct 2015 15:51:07.5298 (UTC) X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.222]; Helo=[atltwp02.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN4PR12MB0851 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: <netdev.vger.kernel.org> X-Mailing-List: netdev@vger.kernel.org |
From: Tom Lendacky <thomas.lendacky@amd.com> Date: Mon, 5 Oct 2015 10:51:03 -0500 > The kasprintf function can return NULL if the allocation fails. Check for > successful allocation before attempting to use the returned buffer. > > Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c b/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c index 66137ff..96f485a 100644 --- a/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c +++ b/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c @@ -327,6 +327,9 @@ void xgbe_debugfs_init(struct xgbe_prv_data *pdata) pdata->debugfs_xpcs_reg = 0; buf = kasprintf(GFP_KERNEL, "amd-xgbe-%s", pdata->netdev->name); + if (!buf) + return; + pdata->xgbe_debugfs = debugfs_create_dir(buf, NULL); if (!pdata->xgbe_debugfs) { netdev_err(pdata->netdev, "debugfs_create_dir failed\n");
The kasprintf function can return NULL if the allocation fails. Check for successful allocation before attempting to use the returned buffer. Signed-off-by: Tom Lendacky <thomas.lendacky@amd.com> --- drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c | 3 +++ 1 file changed, 3 insertions(+) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html