aboutsummaryrefslogtreecommitdiffstats
path: root/src/path.h
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2006-07-30 10:31:41 +0000
committerEric Wong <normalperson@yhbt.net>2006-07-30 10:31:41 +0000
commit263a9d583a74a94fef652bb40f4035b9c0c1612e (patch)
tree52f15e2c2d0e60e87bb1a135c55e4345e2511f2f /src/path.h
parent4144afe551eca63366316c013019bce9ad81bc94 (diff)
downloadmpd-263a9d583a74a94fef652bb40f4035b9c0c1612e.tar.gz
mpd-263a9d583a74a94fef652bb40f4035b9c0c1612e.tar.xz
mpd-263a9d583a74a94fef652bb40f4035b9c0c1612e.zip
remove clumsy strncpy use
strncpy isn't really safe because it doesn't guarantee null termination, and we have had to work around it in several places. strlcpy (from OpenBSD) isn't great, either because it often leaves errors going unchecked (by truncating strings). So we'll add the pathcpy_trunc() function with is basically strlcpy with a hardcoded MAXPATHLEN as the limit, and we'll acknowledge truncation since we only work on paths and MAXPATHLEN should be set correctly by the system headers[1]. file-specific notes: inputStream_http: eyeballing the changes here, it seems to look alright but I haven't actually tested it myself. ls: don't even bother printing a file if the filename is too long (and when is it ever?) since we won't be able to read it anyways. metadataChunk: it's only metadata, and it's only for showin the user, so truncating it here souldn't be a big issue. memset to zero in init is unecessary, so lets not waste cycles [1] - If the system headers are screwed up, then we're majorly screwed regardless of what we do :x git-svn-id: https://svn.musicpd.org/mpd/trunk@4491 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to '')
-rw-r--r--src/path.h8
1 files changed, 7 insertions, 1 deletions
diff --git a/src/path.h b/src/path.h
index 2e8cce241..222e145a5 100644
--- a/src/path.h
+++ b/src/path.h
@@ -23,7 +23,7 @@
#include <sys/param.h>
-extern char *musicDir;
+extern const char *musicDir;
void initPaths();
@@ -51,4 +51,10 @@ char *parentPath(char *path);
/* strips extra "///" and leading "/" and trailing "/" */
char *sanitizePathDup(char *path);
+/* this is actually like strlcpy (OpenBSD), but we don't actually want to
+ * blindly use it everywhere, only for paths that are OK to truncate (for
+ * error reporting and such.
+ * dest must be MAXPATHLEN+1 bytes large (which is standard in mpd) */
+void pathcpy_trunc(char *dest, const char *src);
+
#endif