sbase

suckless unix tools
git clone git://git.suckless.org/sbase
Log | Files | Refs | README | LICENSE

commit 3300d5d47a054bd2615db0f99e1478808ec88032
parent 36e2bdf517773553053571a787ba720bcbfec367
Author: sin <sin@2f30.org>
Date:   Sat,  2 Jan 2016 09:44:06 +0000

ed: Use strlcpy/strlcat to construct the scratch filename

Diffstat:
Med.c | 10+++++-----
1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/ed.c b/ed.c @@ -335,12 +335,12 @@ setscratch() clearundo(); if ((dir = getenv("TMPDIR")) == NULL) dir = "/tmp/"; - if (strlen(dir) + sizeof("ed.XXXXXX") > FILENAME_MAX) - error("incorrect scratch file name"); - strcat(strcpy(tmpname, dir), "ed.XXXXX"); - if ((scratch = mkstemp(tmpname)) < 0) { + if (strlcpy(tmpname, dir, sizeof(tmpname)) >= sizeof(tmpname)) + error("scratch file name too long"); + if (strlcat(tmpname, "ed.XXXXXX", sizeof(tmpname)) >= sizeof(tmpname)) + error("scratch file name too long"); + if ((scratch = mkstemp(tmpname)) < 0) error("failed to create scratch file"); - } if ((k = makeline("", NULL))) error("input/output error in scratch file"); relink(k, k, k, k);