Makefile | 3 ++- noice.c | 14 +++++++++----- strlcat.c | 52 ++++++++++++++++++++++++++++++++++++++++++++++++++++ util.h | 4 ++++ diff --git a/Makefile b/Makefile index 85c46314e9d58a586ffe12f66a659726aa076b97..d8736dfbe050165f2d8799ab65b7943aa6ae76bd 100644 --- a/Makefile +++ b/Makefile @@ -5,7 +5,7 @@ #CPPFLAGS += -DDEBUG #CFLAGS += -g LDLIBS = -lncursesw -OBJ = noice.o strlcpy.o +OBJ = noice.o strlcat.o strlcpy.o BIN = noice all: $(BIN) @@ -14,6 +14,7 @@ $(BIN): $(OBJ) $(CC) $(CFLAGS) -o $@ $(OBJ) $(LDLIBS) noice.o: util.h config.h +strlcat.o: util.h strlcpy.o: util.h install: all diff --git a/noice.c b/noice.c index 16622dd6e6ac904590c384b0884f3bf4aec52282..f52e910f1acefc8759acefc431aca35e6f9dd70a 100644 --- a/noice.c +++ b/noice.c @@ -458,13 +458,17 @@ /* Handle absolute path */ if (name[0] == '/') { path = xstrdup(name); } else { + path = xmalloc(PATH_MAX); /* Handle root case */ - if (strcmp(dir, "/") == 0) - asprintf(&path, "/%s", name); - else - asprintf(&path, "%s/%s", dir, name); + if (strcmp(dir, "/") == 0) { + strlcpy(path, "/", PATH_MAX); + strlcat(path, name, PATH_MAX); + } else { + strlcpy(path, dir, PATH_MAX); + strlcat(path, "/", PATH_MAX); + strlcat(path, name, PATH_MAX); + } } - return path; } diff --git a/strlcat.c b/strlcat.c new file mode 100644 index 0000000000000000000000000000000000000000..979c41bc7ed9c86480ad6dbec733f2eafb314d7a --- /dev/null +++ b/strlcat.c @@ -0,0 +1,52 @@ +/* + * Copyright (c) 1998 Todd C. Miller + * + * Permission to use, copy, modify, and distribute this software for any + * purpose with or without fee is hereby granted, provided that the above + * copyright notice and this permission notice appear in all copies. + * + * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES + * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF + * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR + * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN + * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF + * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. + */ + +#include +#include + +#include "util.h" + +/* + * Appends src to string dst of size siz (unlike strncat, siz is the + * full size of dst, not space left). At most siz-1 characters + * will be copied. Always NUL terminates (unless siz <= strlen(dst)). + * Returns strlen(src) + MIN(siz, strlen(initial dst)). + * If retval >= siz, truncation occurred. + */ +size_t +strlcat(char *dst, const char *src, size_t siz) +{ + char *d = dst; + const char *s = src; + size_t n = siz; + size_t dlen; + /* Find the end of dst and adjust bytes left but don't go past end */ + while (n-- != 0 && *d != '\0') + d++; + dlen = d - dst; + n = siz - dlen; + if (n == 0) + return(dlen + strlen(s)); + while (*s != '\0') { + if (n != 1) { + *d++ = *s; + n--; + } + s++; + } + *d = '\0'; + return(dlen + (s - src)); /* count does not include NUL */ +} diff --git a/util.h b/util.h index 4da38854b9588596c51b3ecd2571e2cc368c041b..89110e51c45f3c04e612b2d21d850f48778a2b40 100644 --- a/util.h +++ b/util.h @@ -1,2 +1,6 @@ +#include + +#undef strlcat +size_t strlcat(char *, const char *, size_t); #undef strlcpy size_t strlcpy(char *, const char *, size_t);