Changeset a21d837 in mainline for uspace/srv/fs/fat/fat_dentry.c

Timestamp:

2013-04-01T16:54:02Z (12 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c8830a2

Parents:

66108658

Message:

Explicitly treat the str argument to fat_lfn_str_nlength() as unaligned.

fat_lfn_str_nlength() was defined to take uint16_t * and as such had
reasons to assume natural alignment of the pointed object. This was
however not the case when the function was called from fat_lfn_size()
because the argument there is an address from inside a packed structure.

File:

• uspace/srv/fs/fat/fat_dentry.c (modified) (3 diffs)

uspace/srv/fs/fat/fat_dentry.c

@@ -44 +44 @@
 #include <assert.h>
 #include <unistd.h>
+#include <sys/types.h>
 
 /** Compare path component with the name read from the dentry.
@@ -232 +233 @@
 /** Get number of bytes in a string with size limit.
  *
- * @param str  NULL-terminated (or not) string.
+ * @param str  NULL-terminated (or not) string. The pointer comes from a packed
+ *             structure and as such is expected to be unaligned. 
  * @param size Maximum number of bytes to consider.
  *
@@ -238 +240 @@
  *
  */
-size_t fat_lfn_str_nlength(const uint16_t *str, size_t size)
+size_t fat_lfn_str_nlength(const unaligned_uint16_t *str, size_t size)
 {
         size_t offset = 0;