Message ID | 1469686136-30713-1-git-send-email-tfchee@altera.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 3s0MlQ0BPgz9t1b for <incoming@patchwork.ozlabs.org>; Thu, 28 Jul 2016 16:44:02 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=altera.onmicrosoft.com header.i=@altera.onmicrosoft.com header.b=Ky4AHuVZ; dkim-atps=neutral Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 46333A768F; Thu, 28 Jul 2016 08:43:59 +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 szz3GbGx3aWZ; Thu, 28 Jul 2016 08:43:59 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id B6A49A7550; Thu, 28 Jul 2016 08:43:58 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id BC0CAA7550 for <u-boot@lists.denx.de>; Thu, 28 Jul 2016 08:43:56 +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 d1oK8pwXfpyK for <u-boot@lists.denx.de>; Thu, 28 Jul 2016 08:43:56 +0200 (CEST) X-Greylist: delayed 2075 seconds by postgrey-1.34 at theia; Thu, 28 Jul 2016 08:43:52 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 NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on0060.outbound.protection.outlook.com [104.47.36.60]) by theia.denx.de (Postfix) with ESMTPS id 2260BA7537 for <u-boot@lists.denx.de>; Thu, 28 Jul 2016 08:43:52 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=altera.onmicrosoft.com; s=selector1-altera-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=E2eFjWkywIkZlDOc1lIRa0cftm9zmIM3GGzT+Yzn9Zk=; b=Ky4AHuVZNj/4VT+XepUGHWUySKZICAPIHMUDdqvJ2yYbY7T1iSB5z8JOjVkslqS4XL8XMDRrcRfyFGYkL1mDMHzDJNpcivX+G1Pp3cAppqOgTq1POMi+55I7HfzD+tLqoVogAzH5BmwWzL2oCDp/wR86i6aclstMMW6AQz7jt9k= Received: from DM2PR03CA0027.namprd03.prod.outlook.com (10.141.96.26) by BLUPR03MB391.namprd03.prod.outlook.com (10.141.78.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.539.14; Thu, 28 Jul 2016 06:09:13 +0000 Received: from BL2FFO11FD063.protection.gbl (2a01:111:f400:7c09::157) by DM2PR03CA0027.outlook.office365.com (2a01:111:e400:2428::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.549.15 via Frontend Transport; Thu, 28 Jul 2016 06:09:13 +0000 Authentication-Results: spf=softfail (sender IP is 66.35.236.227) smtp.mailfrom=altera.com; lists.denx.de; dkim=none (message not signed) header.d=none; lists.denx.de; dmarc=none action=none header.from=altera.com; Received-SPF: SoftFail (protection.outlook.com: domain of transitioning altera.com discourages use of 66.35.236.227 as permitted sender) Received: from sj-itexedge03.altera.priv.altera.com (66.35.236.227) by BL2FFO11FD063.mail.protection.outlook.com (10.173.161.159) with Microsoft SMTP Server (TLS) id 15.1.539.16 via Frontend Transport; Thu, 28 Jul 2016 06:09:13 +0000 Received: from sj-mail01.altera.com (137.57.1.6) by webmail.altera.com (66.35.236.227) with Microsoft SMTP Server (TLS) id 14.3.174.1; Wed, 27 Jul 2016 23:08:49 -0700 Received: from ubuntu.localdomain (pg-tfchee-440b.altera.priv.altera.com [137.57.91.104]) by sj-mail01.altera.com (8.13.7+Sun/8.13.7) with ESMTP id u6S694cU010764; Wed, 27 Jul 2016 23:09:08 -0700 (PDT) From: Tien Fong Chee <tfchee@altera.com> To: <u-boot@lists.denx.de> Date: Wed, 27 Jul 2016 23:08:56 -0700 Message-ID: <1469686136-30713-1-git-send-email-tfchee@altera.com> X-Mailer: git-send-email 2.5.0 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:66.35.236.227; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7916002)(2980300002)(189002)(199003)(81166006)(47776003)(92566002)(81156014)(50226002)(36756003)(356003)(8936002)(19580395003)(50986999)(19580405001)(2870700001)(586003)(8676002)(5820100001)(11100500001)(105596002)(106466001)(5003600100003)(6806005)(2906002)(97736004)(86362001)(305945005)(189998001)(68736007)(87936001)(7696003)(7846002)(110136002)(107886002)(2351001)(16796002)(229853001)(50466002)(450100001)(4326007)(23676002)(33646002)(4001430100002)(7099028); DIR:OUT; SFP:1101; SCL:1; SRVR:BLUPR03MB391; H:sj-itexedge03.altera.priv.altera.com; FPR:; SPF:SoftFail; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BL2FFO11FD063; 1:r083hFct5IoqyXWB8khSq9oa0bwHUe6u6lvcLYhNLM3DVJUj08dMMFArQ69kD9DgDo3GLymLuU9chRM3b9F4Adnbijl0NWZFndYNO5yjqFBlMdGhhkXER3b8COR6FjV/9hfwJGJWox/eod+4xzIzzSSJZZTd+hR1UQITTGJKiN4BU7p4/R8OAY7A9057ZGPqpY8M2l/pxkY65VEmHFHKyBeP8QTf7o6JUStuxDdyO1kbEOvd1Y4/bh/ndjebQVeHMRRESb8m8GgVSuuZ5UY5Aq2U/Wd/I1NKQYAqteXWW7bntvghOvs1+YU36HPZvXVfvFKj+87qj62ilzTX9RT/7M91fiIyTJbVvPe+VZzBA+rHQ1Ruknnlq02Ulzb8ZsM2yU2YBokdGQKQlil3BC+d9cbQOietwrWW7miH5xxqGZXvSaZ/+q6s4op9wJGiSh2QPjEBYIk76hWP6foZHg7DFbwwi8jXJaeHeN7zXo79oCPaoI3USrQBMe45LYTmtGcpYcgp3X3UVjNTZT+MsuaLvPP9oqp736nl2rDgstzIerI= X-MS-Office365-Filtering-Correlation-Id: 38c9f790-caae-4cb1-e4af-08d3b6adb2fa X-Microsoft-Exchange-Diagnostics: 1; BLUPR03MB391; 2:IW6whIfPrSwj5cQJGSd6c81MIT0xfMdSpR1omU9xm5DWJLtrplK8Io5hukNCFLglseo5CctFQDDvAipNArsP/VtITuYhc5vqwlkoAVsO3IzmhvRyYocQUxWELyBzZd0jxmbAC7RiyI1Yn2Dwjwup34+5vb0hDF4f2c4N3/BdLRcDAgbvm4wig3JIsaK8DQJh; 3:Kq2AJMTSNri1GpW++ov2DI32pF0KMuEn0pNdN5+JEBO/iX6zCUX+rajCWB9YQHcIo2/dqzgZbnvxIFPwZBwDuGnoETIvXZbtEpHubTc01IavIn3B8Sx+ZbDI9+IXkZMuJYTziiMWidPNG+85JAQSMD8iBLkm3Jmjd1yiXlD5gV9m7lBkO7cQchpvohsJYkmZcfxgZwmFNw2qq2yL+dvckpS8R236vMXq4NvgRMIrO5g=; 25:I8pmfVzYt0uh+IEU2Ssset0agksB5khKbSvpUHYwAGeIcpcOPKoyp0Z4FcdG9Lxf/MlzfM8B+0LxLZg8nyC6Pr3NFuCznf+oRuU9mBUN54V5y/hYRO+lTHaKFPrYawCIC9/Z//ma4AdaejdMMvo6GCP19eg0Wch3pRt61t6dk5UPvACTiU/TCNQY7ymVE4FIKx6jO2qfFgZAbgFjyjgAZkEanquGnb/fxTFwV0BJzmRBQvAv2SmEe93BFJxeOnvKWB5Gx+XdwlzlACbhuhhmoDEcKLeVIywtsEsyqrIyabo6ntXEsabJEylHPHf84sf1Ep4hh0i/SGuro7KnC8y9p37c+QcbconDWfYcp9NOI6upqYU1R3Rs1wK1Kiq//daCnG/HLlfM0Qh2fsDPE5XlN2qvqqlutpsmcfFmry+uMxY= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BLUPR03MB391; X-LD-Processed: fbd72e03-d4a5-4110-adce-614d51f2077a,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; BLUPR03MB391; 31:LKlTBssD1li8A9vMz0IgwexNoY4kbGGTpI0g0rgH+f1y56opasuhj7ZR+DIApsA8zHdl3u8z6fxz1ePOO9yMwI9ZdsO1bJfeaWo1sC5j61Zvwn4qgl4RYq7CZ4Lq54XF0b+/G2fP3/TNadosodrDvFtADWRZYI5Hdvj6yYjL8CCajzFEpIlcQvjNrOeu3bKaX3V2BpO4mMpTxFRwvs5OhQ==; 20:NlRciDCD+3O3Cvrz5bVEo8Ti/hwyZUdXvZ9YX57MlHe4b/hvOygIfDRxqrEeamKbMHro5Ofjqr2kvf4Pie5XOCGOpZyILfvM0YLl2xwIQKCK4t8uGErzhRWGx12PCSvLtMWOKpMB2w7S5Ho2mw9s79skfPTDnts9pVk4uKeN7yo=; 4:ieR30txhHG68Xt0Be0FzXH9qpX3/psEhZs/xFv0tOMUpnHjqQNfarBfo2MZoqDo5CJd8UhXhDaSGLHVOOvGNbSsgL7veu3njuM1AkckGWzWILvjrfGLv3paeL8lyZIabMuJntx+DN/d8ysWBDmzafgb9wSQemEyShKYJ8UvGR7Z+upEouVQpenKCRJrV/p8KY2PZVP5f4WzLUxoFMzMpJqDrbvczfHQxCdnJEcXDphdyczehYAlFwnU8C1f0RLaS+yTILvW2u+X2XqLY2XmkE5ucE45ANhZqNlki7tiBMaDFstAUjYD1zSMBGmD/VZA2oDZz12UyfmhQ99jUEa+jlZ4P4MP6WZGUd/JXAu8O3WmFfOvoKNAAFJpmglrBnTKHvT0Nd8W847msMqjoqbUC5eQBlJnsJ35aO4FF1fFu89GGT/meY+4SxZpbH3Zsk+q9nczVV4euRnyLh2/QbZNIRsdO2WzffSSRnBoJNH3PL4p6WR0bscprW36+OdpF5d0AdemJyDL1RHU3iB2IKaPdxA== X-Microsoft-Antispam-PRVS: <BLUPR03MB3919ECD8EA183A09A4FFFFFA3000@BLUPR03MB391.namprd03.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:(80048183373757); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(8121501046)(5005006)(13015025)(13018025)(13023025)(13024025)(13017025)(10201501046)(3002001)(6055026); SRVR:BLUPR03MB391; BCL:0; PCL:0; RULEID:; SRVR:BLUPR03MB391; X-Forefront-PRVS: 00179089FD X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTFVQUjAzTUIzOTE7MjM6eVljSTUwYVgxN3pHYWszLzhtVWZxUTU2V2tH?= =?utf-8?B?TFE5N0l0czJEWm1tU1hidFl1YUYzRjVFNG4vc2g5cFdkS0dySEFtdVYzeTZB?= =?utf-8?B?dDA1bTRQZGRmanVXQTZBbmE3cDhxN2hRYytXYkdwVU5sZXU0SURsUk1FT2My?= =?utf-8?B?SlZna051ZDVVWCtHQW5JYm5oc2RpVE1hTTY2Q3RsSTJ3VU9CYThsVThKdExn?= =?utf-8?B?OFhCaTdPV29TY25XSXpKZTRqL3R2NGZDbG1lZlQxOW1XdVorSUFmaHFlR3FU?= =?utf-8?B?NnRaZjhIaVZ6WUx5cDFUVmsyY3loNHZFaUYzWlNONUxQRWRGNHJNTzNNZmxr?= =?utf-8?B?MjE5MVhnSnJNeTk4ZHFScm13RjZBdVV2MVJkVGJRemFPNTFXQW9Sb2J0dVNC?= =?utf-8?B?NEM0bitnLzRyVWRGNjRhcHRYUGFqeW54TVYwaEZGR1dFVWhHc0hJd3hPNmN4?= =?utf-8?B?b1FOSUp2bWVoQjZqSGR4K3UzMG05SVJhZHdxYlZRK0dvV1hoN1JMbGV0bTIy?= =?utf-8?B?TE40QzlzN3JVb25WRTNycGRZbSs0bGMzLy8vNEI1Vkh5a0hRWWxsekVlSWl1?= =?utf-8?B?ZmU0YTFjVlFGdjhkMXdZY01FRG10MWptSGVEMDBJYW8zOGdiaDRkV1kvcDcw?= =?utf-8?B?THpyNFZkK0ZWYzBQaGVuZ0RGMEJkVHFhL0VRUmdxSDI2Z0lwTERSTFJyYTk4?= =?utf-8?B?MXREMnhiVzFuMm5SaU1CdUs1ZFBEUkp2alcydkZpYWJGaW05eWxXNy9DU2Y4?= =?utf-8?B?SEZWYU5mVXc0eDZtKy9PVVdybEd1YnhDekNpWlIybmordUE3bFVNZmhVLzRh?= =?utf-8?B?ZXlzK0s1d2tkMmVxUGVDR1pjVjR6MG11eE5BNGlQOEorb0xNbkgzNXV0S01u?= =?utf-8?B?Y0xPNDRwNXRyeEJ4blNDYm1weDZOcms2T3FLK2JxeXlCNTBobTFPbUIybVZS?= =?utf-8?B?TXpXZG9QbTh4aFZ4QXRxVzNxejZOUVBWaHBrUEJOUW1heHJEa1Vtb0hLTmtu?= =?utf-8?B?NmZlQTZKYVFDcGlkODlMRTlHRnh3R29ONTlBMk1vcUptOFdjZzZMUmE5VVNV?= =?utf-8?B?MjRObkNoV05saGM3aUtYUVM3SnlUdU1ibnBsTUVCWXpOODZZcG9nRDNQUmd6?= =?utf-8?B?ZXI5eS9rSFRsbTJteXRNS09QeDJ2RlFudTNGaTY2L1hLdXhlOTFWQzZOMnVk?= =?utf-8?B?Tmh4M2JGN0hobU8vaExWaDVjNVJXdjVXbTVLT1EvcXRQbHQ4YXd1RTM4WDNF?= =?utf-8?B?UDA2Z09WTmsraCtFNm53KzRGRmMwcHlHdGM0TWhzZzY5ZGdieVM3UWFwcDN2?= =?utf-8?B?ZlpiWmJEZUdHY2ZlSTlEZ1RHZ3AreFd2OE04dEgwMWhZQXowQTdKVXdIdkFa?= =?utf-8?B?aTllNURMbnYxOUtraDJKOVZPRXhEaEJUTHJMMXpNZjJsRDdMNm5IOEpYbTQr?= =?utf-8?B?TTQwSnE5THoyWGI0SEpWRXJHbDNNcWhTb2d3Rm9YSUVvWitndEhjSXJDN3VN?= =?utf-8?Q?0wem68AszNyIcI2RQYv8TjgI=3D?= X-Microsoft-Exchange-Diagnostics: 1; BLUPR03MB391; 6:rMvhlDWeZHu691lOcW7cK14jU+AayNPqAsfBtHt3NemEU1LW8hiucA5TxqKgbx34jKITjgzLdlWyeFv98Vd5Ep6YjsWkDn/o7mprQhih4ky0v5kVxbk2Z8ybRvyb13vrixB68QjqUQvUIvlQG0c1MISp3QLbEx2CVFAmZyAGOGZvwYVkrq4BdZpiRnI3Sg1PWt9JRCpbRO6AaJBc/f7IT4RI6rvHdGoTzFZ3E5FUZn29FvjQ34rW2t+zKBEEVB42C1pLrf/9yUAhnW/E5bihpHyRPIZq7Xv1/jH2Iavhh0JvTFp8qWwGlg8TP3sDCxfrf3h0hdI+IcP/H9QgSYz1Nw==; 5:HNRxgR0PuGfdaVgdct+4xg3/+dXLfoWXVchbe0L0wPEMYbcOMV488hS3wy83EZuG9FpPI47iuFS4uHWIuFw0FlKFLyFTim9wd/uAlI37j61xFgCyNYUtciq9o34qRWJlOweSftKILf/D4VJ6uCooRA==; 24:V9eHjr4SjQxBeidfJ4suI4miZaSiUudcBrpdu2Y4w+NuCP3sihH3tCZ+7p2Y690YBuMwU0MIPlb0stgbWxwJzwyiGh6lpARBp08sahYSpWY=; 7:xhpkoZJZEfSScLhhxvC04S712N3hM8aCQBb+mjSnEn2PQL+sY/DwM3ZQfIGdpsL0D0s9TUUYH4WS7HSSkhOM3c5MguokK2SJZfPCdU5CuAAFXtM5NSil8FPGxIHylCwEo8KwQ/nkGuYYnu2JRUgj0F9a5v3NQKWD5vOUjnP9WZc34FKkoci39FWvoFRfTJ/DE59aMip/b4LtAEmzLyykNp+Gu0hM1Tybe6H5BhsEI3cDyZb3feiLale8bQfSAZjz SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BLUPR03MB391; 20:fbNEwnj9Q2cKPja6mPMuqur3PVNO0b5UWEs0MHZE33zhfnHvLQBvh6rLdK67ULRYG9amaqtZmAligNKlf8D+fkJrOcxZiA8mFUMf99Zs7W8b1f4C+Kz3ER8b1iIiDsjcZX5YtwNLs49wjq+1jWicJATIXDphNwIn5Ae1BO67yW4= X-OriginatorOrg: altera.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Jul 2016 06:09:13.4480 (UTC) X-MS-Exchange-CrossTenant-Id: fbd72e03-d4a5-4110-adce-614d51f2077a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=fbd72e03-d4a5-4110-adce-614d51f2077a; Ip=[66.35.236.227]; Helo=[sj-itexedge03.altera.priv.altera.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR03MB391 Cc: Vagrant Cascadian <vagrant@debian.org>, ChinLiang <clsee@altera.com>, Genevieve Chan <ccheauya@altera.com>, =?UTF-8?q?Beno=C3=AEt=20Th=C3=A9baudeau?= <benoit@wsystem.com>, Stephen Warren <swarren@nvidia.com>, Tien Fong Chee <tfchee@altera.com> Subject: [U-Boot] [PATCH 1/2] fs/fat/fatwrite: Local variable as buffer to store dir_slot entries 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, Jul 27, 2016 at 11:08:56PM -0700, Tien Fong Chee wrote: > fill_dir_slot use get_contents_vfatname_block as a temporary buffer for > constructing a list of dir_slot entries. To save the memory and providing > correct type of memory for above usage, a local buffer with accurate size > declaration is introduced. > > The local array size 640 is used because for long file name entry, > each entry use 32 bytes, one entry can store up to 13 characters. > The maximum number of entry possible is 20. So, total size is > 32*20=640bytes. > > Signed-off-by: Genevieve Chan <ccheauya@altera.com> > Signed-off-by: Tien Fong Chee <tfchee@altera.com> Applied to u-boot/master, thanks!
diff --git a/fs/fat/fat_write.c b/fs/fat/fat_write.c index eb3a916..c1d48c5 100644 --- a/fs/fat/fat_write.c +++ b/fs/fat/fat_write.c @@ -323,7 +323,8 @@ static void flush_dir_table(fsdata *mydata, dir_entry **dentptr); static void fill_dir_slot(fsdata *mydata, dir_entry **dentptr, const char *l_name) { - dir_slot *slotptr = (dir_slot *)get_contents_vfatname_block; + __u8 temp_dir_slot_buffer[MAX_LFN_SLOT * sizeof(dir_slot)]; + dir_slot *slotptr = (dir_slot *)temp_dir_slot_buffer; __u8 counter = 0, checksum; int idx = 0, ret; char s_name[16]; diff --git a/include/fat.h b/include/fat.h index 9d053e6..483ff7c 100644 --- a/include/fat.h +++ b/include/fat.h @@ -33,6 +33,8 @@ #define FAT16BUFSIZE (FATBUFSIZE/2) #define FAT32BUFSIZE (FATBUFSIZE/4) +/* Maximum number of entry for long file name according to spec */ +#define MAX_LFN_SLOT 20 /* Filesystem identifiers */ #define FAT12_SIGN "FAT12 "