about summary refs log tree commit diff
path: root/include
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-06-06 18:04:28 -0400
committerRich Felker <dalias@aerifal.cx>2011-06-06 18:04:28 -0400
commitda88b16a221c9d327e1bfa61dd6f4f08dacce57a (patch)
tree64d07cc9ac0fc99dc6026d188f01943e81c055a4 /include
parent0dc99ac413d8bc054a2e95578475c7122455eee8 (diff)
downloadmusl-da88b16a221c9d327e1bfa61dd6f4f08dacce57a.tar.gz
musl-da88b16a221c9d327e1bfa61dd6f4f08dacce57a.tar.xz
musl-da88b16a221c9d327e1bfa61dd6f4f08dacce57a.zip
fix handling of d_name in struct dirent
basically there are 3 choices for how to implement this variable-size
string member:
1. C99 flexible array member: breaks using dirent.h with pre-C99 compiler.
2. old way: length-1 string: generates array bounds warnings in caller.
3. new way: length-NAME_MAX string. no problems, simplifies all code.

of course the usable part in the pointer returned by readdir might be
shorter than NAME_MAX+1 bytes, but that is allowed by the standard and
doesn't hurt anything.
Diffstat (limited to 'include')
-rw-r--r--include/dirent.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/dirent.h b/include/dirent.h
index ca000bd9..7b70abd4 100644
--- a/include/dirent.h
+++ b/include/dirent.h
@@ -18,7 +18,7 @@ struct dirent
 	off_t d_off;
 	unsigned short d_reclen;
 	unsigned char d_type;
-	char d_name[1];
+	char d_name[256];
 };
 
 #define d_fileno d_ino