Commit 9a02f8b79138972ddde059e5683108d07b409289
2020-12-21T12:59:27
switch to strerror_r(3) in error.c for thread-safety
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60
diff --git a/lib/error.c b/lib/error.c
index a41a721..72e0048 100644
--- a/lib/error.c
+++ b/lib/error.c
@@ -87,9 +87,10 @@ got_error_from_errno(const char *prefix)
{
struct got_custom_error *cerr = get_custom_err();
struct got_error *err = &cerr->err;
+ char strerr[128];
- snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s", prefix,
- strerror(errno));
+ strerror_r(errno, strerr, sizeof(strerr));
+ snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s", prefix, strerr);
err->code = GOT_ERR_ERRNO;
err->msg = cerr->msg;
@@ -101,9 +102,11 @@ got_error_from_errno2(const char *prefix, const char *prefix2)
{
struct got_custom_error *cerr = get_custom_err();
struct got_error *err = &cerr->err;
+ char strerr[128];
+ strerror_r(errno, strerr, sizeof(strerr));
snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s: %s", prefix, prefix2,
- strerror(errno));
+ strerr);
err->code = GOT_ERR_ERRNO;
err->msg = cerr->msg;
@@ -116,9 +119,11 @@ got_error_from_errno3(const char *prefix, const char *prefix2,
{
struct got_custom_error *cerr = get_custom_err();
struct got_error *err = &cerr->err;
+ char strerr[128];
+ strerror_r(errno, strerr, sizeof(strerr));
snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s: %s: %s", prefix,
- prefix2, prefix3, strerror(errno));
+ prefix2, prefix3, strerr);
err->code = GOT_ERR_ERRNO;
err->msg = cerr->msg;
@@ -131,13 +136,15 @@ got_error_from_errno_fmt(const char *fmt, ...)
struct got_custom_error *cerr = get_custom_err();
struct got_error *err = &cerr->err;
char buf[PATH_MAX * 4];
+ char strerr[128];
va_list ap;
va_start(ap, fmt);
vsnprintf(buf, sizeof(buf), fmt, ap);
va_end(ap);
- snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s", buf, strerror(errno));
+ strerror_r(errno, strerr, sizeof(strerr));
+ snprintf(cerr->msg, sizeof(cerr->msg), "%s: %s", buf, strerr);
err->code = GOT_ERR_ERRNO;
err->msg = cerr->msg;