From: Tom Rini Date: Wed, 17 Oct 2012 08:03:24 -0700 Subject: [U-Boot] [U-Boot, V2] fs: fat: Fix mkcksum() function parameters In-Reply-To: <1349803222-15917-1-git-send-email-marex@denx.de> References: <1349803222-15917-1-git-send-email-marex@denx.de> Message-ID: <20121017150324.GG5527@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Oct 09, 2012 at 07:20:22AM -0000, Marek Vasut wrote: > The mkcksum() function now takes one parameter, the pointer to > 11-byte wide character array, which it then operates on. > > Currently, the function is wrongly passed (dir_entry)->name, which > is only 8-byte wide character array. Though by further inspecting > the dir_entry structure, it can be noticed that the name[8] entry > is immediatelly followed by ext[3] entry. Thus, name[8] and ext[3] > in the dir_entry structure actually work as this 11-byte wide array > since they're placed right next to each other by current compiler > behavior. > > Depending on this is obviously wrong, thus fix this by correctly > passing both (dir_entry)->name and (dir_entry)->ext to the mkcksum() > function and adjust the function appropriately. > > Signed-off-by: Marek Vasut > Cc: Tom Rini Applied to u-boot/master, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: