[ros-diffs] [navaraf] 33447: Fix wrong variable usage - "Filename" (8 characters long) of a directory entry was referenced instead of a "ShortName" (11 characters long). The actual bug was harmless since both arrays are defined in an union and lie at the same place in memory.

navaraf at svn.reactos.org navaraf at svn.reactos.org
Sun May 11 20:57:07 CEST 2008


Author: navaraf
Date: Sun May 11 13:57:07 2008
New Revision: 33447

URL: http://svn.reactos.org/svn/reactos?rev=33447&view=rev
Log:
Fix wrong variable usage - "Filename" (8 characters long) of a directory entry was referenced instead of a "ShortName" (11 characters long). The actual bug was harmless since both arrays are defined in an union and lie at the same place in memory.

Modified:
    trunk/reactos/drivers/filesystems/fastfat/dirwr.c

Modified: trunk/reactos/drivers/filesystems/fastfat/dirwr.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfat/dirwr.c?rev=33447&r1=33446&r2=33447&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/fastfat/dirwr.c [iso-8859-1] (original)
+++ trunk/reactos/drivers/filesystems/fastfat/dirwr.c [iso-8859-1] Sun May 11 13:57:07 2008
@@ -386,7 +386,7 @@
         {
             pSlots[0].alias_checksum = (((pSlots[0].alias_checksum & 1) << 7
                                      | ((pSlots[0].alias_checksum & 0xfe) >> 1))
-                                     + DirContext.DirEntry.Fat.Filename[i]);
+                                     + DirContext.DirEntry.Fat.ShortName[i]);
         }
         /* construct slots and entry */
         for (i = nbSlots - 2; i >= 0; i--)



More information about the Ros-diffs mailing list