Message ID | 1444225728-23057-1-git-send-email-vincent.stehle@freescale.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
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 7E886140D88 for <incoming@patchwork.ozlabs.org>; Thu, 8 Oct 2015 01:13:29 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id A23E84B7F9; Wed, 7 Oct 2015 16:13:14 +0200 (CEST) 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 MZVy43xkyj5x; Wed, 7 Oct 2015 16:13:14 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 182864B803; Wed, 7 Oct 2015 16:13:10 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id E704C4B803 for <u-boot@lists.denx.de>; Wed, 7 Oct 2015 16:06:21 +0200 (CEST) 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 jJRllDZCSlpY for <u-boot@lists.denx.de>; Wed, 7 Oct 2015 16:06:21 +0200 (CEST) X-Greylist: delayed 923 seconds by postgrey-1.34 at theia; Wed, 07 Oct 2015 16:06:18 CEST 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-bl2-obe.outbound.protection.outlook.com (mail-bl2on0116.outbound.protection.outlook.com [65.55.169.116]) by theia.denx.de (Postfix) with ESMTPS id 3F4274B7F6 for <u-boot@lists.denx.de>; Wed, 7 Oct 2015 16:06:18 +0200 (CEST) Received: from CH1PR03CA005.namprd03.prod.outlook.com (10.255.156.150) by CY1PR0301MB1306.namprd03.prod.outlook.com (10.161.212.16) with Microsoft SMTP Server (TLS) id 15.1.286.20; Wed, 7 Oct 2015 13:50:53 +0000 Received: from BN1AFFO11FD023.protection.gbl (10.255.156.132) by CH1PR03CA005.outlook.office365.com (10.255.156.150) with Microsoft SMTP Server (TLS) id 15.1.293.16 via Frontend Transport; Wed, 7 Oct 2015 13:50:52 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=freescale.com; chromium.org; dkim=none (message not signed) header.d=none; chromium.org; dmarc=none action=none header.from=freescale.com; Received-SPF: Fail (protection.outlook.com: domain of freescale.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 BN1AFFO11FD023.mail.protection.outlook.com (10.58.52.83) with Microsoft SMTP Server (TLS) id 15.1.293.9 via Frontend Transport; Wed, 7 Oct 2015 13:50:52 +0000 Received: from laptop-vstehle (laptop-vstehle.ea.freescale.net [10.161.68.165]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id t97DophI011810; Wed, 7 Oct 2015 06:50:51 -0700 Received: from vstehle by laptop-vstehle with local (Exim 4.86) (envelope-from <vincent.stehle@freescale.com>) id 1Zjp7K-00061m-C0; Wed, 07 Oct 2015 15:50:50 +0200 From: =?UTF-8?q?Vincent=20Stehl=C3=A9?= <vincent.stehle@freescale.com> To: <u-boot@lists.denx.de> Date: Wed, 7 Oct 2015 15:48:48 +0200 Message-ID: <1444225728-23057-1-git-send-email-vincent.stehle@freescale.com> X-Mailer: git-send-email 2.5.3 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1; BN1AFFO11FD023; 1:xJneA5BqwB5aZB0f1WBYgLiIjcyWxbtCtmRtTQSGrmDPRAtx5Ni4Hrhf3f6tuc/meucBWo6Qr/t16xoms9k23RAzTmKlYcXI/ZbWEXSZFDEwHX548k+tG7eR1zq6yU8QggoM8FWrpqUXD3Z1jQbf3rXDNYbCs/bJOxy49Cl9gfVfV2xp0nWX3ly4Eo2xaRHM9eOQLsnsvCTro6e+bkRM7GvSEsrLKKmznc+DM6wB6Tu8ke38g3l9EfUzmvOQE6kIzRqrYHNsY2zAJmQdGt+d0Pn5wmv6kVq4JqS2dVXxMXY8o8T5SZvNYgQIs1MUU6xspttVemsko6znSxZFX+pP7/yCNKyVvq4CTQ53mSeTbzToAZo+vKMuBgZjma55B96H33WrWpJfwEBcTKzMLngFXg== X-Forefront-Antispam-Report: CIP:192.88.168.50; CTRY:US; IPV:NLI; EFV:NLI; SFV:NSPM; SFS:(10019020)(6009001)(2980300002)(1109001)(1110001)(339900001)(189002)(199003)(57986006)(5008740100001)(6806005)(81156007)(97736004)(92566002)(5007970100001)(16796002)(36756003)(33646002)(50226001)(23676002)(106466001)(229853001)(19580395003)(87936001)(110136002)(11100500001)(5820100001)(64706001)(19580405001)(105606002)(50986999)(46656002)(189998001)(85426001)(2351001)(47776003)(5001960100002)(46102003)(50466002)(86362001)(83796002)(230783001)(70036007); DIR:OUT; SFP:1102; SCL:1; SRVR:CY1PR0301MB1306; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; CY1PR0301MB1306; 2:Bqku4EeSG5iF8/g671Wc29BdHJ6sVuUvjzamlxU4P7z12jXREQa0/cvFZSv4cnGvwJOTC0ziaT/Hyowi/kmmZNkeTACMhA/8hEWUbIIpdfsoKLFK3Led6ENdX/1RHcWc3qYuCDw9AKP/9ObmrkelLXeCPxpv45f8B42w8ZcWxzg=; 3:ZbMhzkUsypMzyZ2645D157y1TVFzyavVNpqg79HdLU7PE+q1JVHvlshH98fuFcKY9nxmdyFsRddh0PdGpsfd/YaVAsUtkDR87rxGLShPfOtMvPfUWSq3M/k0IY5YUJTtXdAEQA8cdWApcqjJ7E6Wrq5bAWIVgwa3e69NiIEVV5naf+kp8sFxf8enyMOY2hmY1W+uy9cO3dBeIBXTT747P/8OgVPCXlsfzx1qRkucZU4=; 25:PbPRFHyed8DsDD2ziG7tQHxfWKUQh7z1bnzS/K6R8uEZvBiGLoO+D/9EVD8EtdNkne/7XzwKjtbt6wSQ+3X9yZ2dGcHxkE21Hb1WGtJq210bfLaYHjZe9q6QlBXXfhkmnNvPpekjP19+5wdPphfr37TtUD2MjUH/U5AgppJuVHpP4ggA4JIriYbxAdGvhmlvJRyCK/cuVOPZH/tiM+S3H58P3pkexJ59WMbz6T7CCvNNDEIDgEVOiYYk1OrtJyOxAkGvNYUN8S4SjrNRBYpS9A== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:CY1PR0301MB1306; X-Microsoft-Exchange-Diagnostics: 1; CY1PR0301MB1306; 20:M4SpOKWKf58jugbuXND3BmYTR1axalvpNkW7GL2D5dtZ2uTaC/H/YziB/iA6LyRONt7Sw5uCX5T1JWzzahv96jxCgL6ahzP0p0QWd28TIdpgl+8E95FmXX3UFJx5uHdAlzaGDDwr445G9/qsA3rGulPzrluD2rrfs1RLdYH+6/cjva+uUzimgn6cLUa+Q+MaenSuY2+aXvDU5S8xy+7dY+/kr9C7IOuYQw86vKt+1OdMxDzN1PBeRWsi+nTPMfZbEzsvYTbNmtSuAG/57G5JKKE/J+Od2eCPefEtLwuyPrpvxXA7uz8shll1WRF64IFM+kEHXPlHTyfvVbbXs8foXgiBbdSjeqJQ3sSqhwOx/vw=; 4:7MiMJH41nDqjURYCAbnx4ZfDdmm+jrLQmBlK1jdgKnMf0RMXUTecq50LlA8NG/cvnZRbvebG8jrEemnVbqJwvxQ2VlxfxTFwv96D7tH7mC8OU3IiqAGPOJxL+HPBel/MxIsVsnVhYXRyCR4GnLFZAyhqeAXG61c6Gq56d04WSvupZ0TdbKh0alukAmYRmRwW7OLoPx28je0GATfDJ8S7hG/JwsLYOZe03ZeLYncdqUPoTQ85QW4ShclEiiIJ8hn+61ldbKxtS+4z8XNeUEgBFOHLnb7zgtgtP2l6u6W5M/ydG9s5LiqaYKdhEpiuvfEC0jQ+mC5oCkxeWpIL/raKE6qOCUWxwMbpME+sRrmxH7c= X-Microsoft-Antispam-PRVS: <CY1PR0301MB1306A4F2CFAFFC4E1A979D7099360@CY1PR0301MB1306.namprd03.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:CY1PR0301MB1306; BCL:0; PCL:0; RULEID:; SRVR:CY1PR0301MB1306; X-Forefront-PRVS: 0722981D2A X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtDWTFQUjAzMDFNQjEzMDY7MjM6cXhzRXA0ZmRoLzV5QmYyZ1UwSjM0elZ3?= =?utf-8?B?YjdzNmJ0b3J5QmNDL2QrSGhWN2tPR1duQm1FYTFSLytWdW5lZXhDNEJoZDNn?= =?utf-8?B?a1pHdFpONkxIWnNrc0ZXMkdCSkpyS0Nud3haSHVOb3hWQjgrWXR0Z1QvRmFk?= =?utf-8?B?YlF6T2lRZGVQazF0V2o0UHVmWmtFOTk1ZzZsblNDY2kvTzVyNkF0VytqNjJi?= =?utf-8?B?SXJwYkdFMTg3WnRXZzIxMGJXd1pBVjJwS2pIVnFkcDFlTmNWRWJlc0orWE8w?= =?utf-8?B?TENmZXB1YWFUOWtmOUYvVG9qSzdCemRnL1ZLSWFkUDJFRmRaYUJTZHd5MnVU?= =?utf-8?B?NytndVhPVE0vRFROQjdic2wySDdmV2JZd3hIeFd1R1pQdFFGVFNKY1JWM0dZ?= =?utf-8?B?a05aUTYrVmZkK0VsZXBNWWZ1T1N3TU1RejBvTWVjVGxZYkdnRDczdXFLYUxW?= =?utf-8?B?d2VidTZDeWMrMmN6bCtPVmRFMExLckxUTnVwdFJoRDF6bjdxak9uYnZLUnBi?= =?utf-8?B?bHZGTUErTFVvd09ORVdsYmdudFpIVnhCMUJEMENGZ1lhcnFEZEo0SmlqZVkr?= =?utf-8?B?NGxPQnYxTjI3TjVGRzlnRHpYVjBlb0FUWGgvUG56aGVhdnJaUnlQUWMrZi9i?= =?utf-8?B?bXVOS2xVVU5MY1BieHdrWUpETEdaRGFZMzFVbXNkeGlkVmRMbXZjQ0owc2h6?= =?utf-8?B?NnlBWGVpT0FMVkZCeldPYjRYNi8rMUowTGpkU3ZCUkZGVVBkSVhCb0liMy8y?= =?utf-8?B?bWQxMktoaVFqZzhZRXBweXZ5cGM5YU1OQ0thQmZpcEtqc2hMRGtXajlzb3R0?= =?utf-8?B?N3YrdzU1Z1MrU1ZqcXV5L3FGd3RSNUtoVFVoU0F1WHkwb1ZqZVdSeUt2YzRk?= =?utf-8?B?bjNHd2phd1BDTjliRW9NVEZLQkZXYVMxUk05azBCeklBY25lN3lQQ25xSGFT?= =?utf-8?B?ZEY1TDFGSWxaNnZxaFhWSkExdlRqd3NPUXRMOE9FSENINGNJWk81SElQOXFH?= =?utf-8?B?elh1cGJ0eFRDWHBYTEVFa2VOTStFL3UzRGZoUGJWNDM2bkNrb2NOV1JzdmZO?= =?utf-8?B?SGFaeVBDWi96Um9HSUdFdndtRWttazVSNW0yaU1mY2lrYzY5N1Ewc0pwYVAx?= =?utf-8?B?NnNqU2ZqVlk5R2VOcS91R0x3UGg3TkwyM0tEd25NMHFnY3ZGSTh0bktKNER2?= =?utf-8?B?QjZ1cXhSNVN0cDBZUzE4eCtENnVUdE5Gam45NU53YmhwWUQ0Mmt1aWRaTVE1?= =?utf-8?B?S0V2bVRpQmxXdE93SzkwdXRTQU50Q2dtaXBKQVIydmJTTnVCb1JrZy91NEdV?= =?utf-8?B?MFBDWit4dzY3cFE2dG4yeGFUZmJhdTdBLzZXM2NudURhRUxtWEJ3Z3dLOVFK?= =?utf-8?B?dWhrN29VT3hqSzZxb2t3MnhTZlNGQkMyQkNOeEc5aERyMjRxalhIVnVaeWl5?= =?utf-8?B?Z05ZRVNPK2wyWmFsZUxsY0xpUjNVVEFVVmpmMmF3UmViSEpjR2pyOWJEUUg2?= =?utf-8?Q?ExSRetgMjnfu9f3coAZUk14pKro=3D?= X-Microsoft-Exchange-Diagnostics: 1; CY1PR0301MB1306; 5:ZmYf4VWO3UM1Yc6J6VRt5bKEYPovPXPwAwe0jWkBD//1KUk9PTX8/ZGGsAc88jNKGvHChISOHzCacTmuaFQlR/5NHMeArmRfc37+WmlGFvJkBkUQOcDNu1uDe1JEigne0EUHn4zhQDnaTmkCJDNaGA==; 24:6S/bPLMwWkdXiBkz63lco2xms8EpG2ztSEoForAo0ttZgNKH1+sVDBP+1DAZ0Vkf/fQ1jTqMhTMqH2FZ1EU3k6W//T1rrKb0T6TQLTBwXQw=; 20:KKkZz2F3yL7Luf6g0H5e+ab+1v0+YywvkJUIhrYauFJ88zYmB5TUIgcqIdIUquel5hEEsQlPD4ZSmEaIIlWMrA== X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Oct 2015 13:50:52.5414 (UTC) X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR0301MB1306 X-Mailman-Approved-At: Wed, 07 Oct 2015 16:13:06 +0200 Cc: =?UTF-8?q?Vincent=20Stehl=C3=A9?= <vincent.stehle@freescale.com> Subject: [U-Boot] [PATCH] tools/proftool: fix use-after-free 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="utf-8" Content-Transfer-Encoding: base64 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de> |
On Wed, Oct 07, 2015 at 03:48:48PM +0200, Vincent Stehlé wrote: > The read_trace_config() can dereference the line pointer after freeing > it on its error path. Avoid that. > > This was found by Coverity Scan. > > Signed-off-by: Vincent Stehlé <vincent.stehle@freescale.com> > Cc: Simon Glass <sjg@chromium.org> Applied to u-boot/master, thanks!
diff --git a/tools/proftool.c b/tools/proftool.c index 9ce7a77..ddf870f 100644 --- a/tools/proftool.c +++ b/tools/proftool.c @@ -432,9 +432,10 @@ static int read_trace_config(FILE *fin) err = regcomp(&line->regex, tok, REG_NOSUB); if (err) { + int r = regex_report_error(&line->regex, err, + "compile", tok); free(line); - return regex_report_error(&line->regex, err, "compile", - tok); + return r; } /* link this new one to the end of the list */
The read_trace_config() can dereference the line pointer after freeing it on its error path. Avoid that. This was found by Coverity Scan. Signed-off-by: Vincent Stehlé <vincent.stehle@freescale.com> Cc: Simon Glass <sjg@chromium.org> --- tools/proftool.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)