From 472e8b71f7a90ab23f1499fd721b872541f52de9 Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Thu, 15 Jan 2015 07:09:14 -0500 Subject: for multithreaded set*id/setrlimit, handle case where callback does not run in the current version of __synccall, the callback is always run, so failure to handle this case did not matter. however, the upcoming overhaul of __synccall will have failure cases, in which case the callback does not run and errno is already set. the changes being committed now are in preparation for that. --- src/misc/setrlimit.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/misc') diff --git a/src/misc/setrlimit.c b/src/misc/setrlimit.c index 8a1b8cc6..7130d719 100644 --- a/src/misc/setrlimit.c +++ b/src/misc/setrlimit.c @@ -32,16 +32,16 @@ struct ctx { static void do_setrlimit(void *p) { struct ctx *c = p; - if (c->err) return; + if (c->err>0) return; c->err = -__setrlimit(c->res, c->rlim); } int setrlimit(int resource, const struct rlimit *rlim) { - struct ctx c = { .res = resource, .rlim = rlim }; + struct ctx c = { .res = resource, .rlim = rlim, .err = -1 }; __synccall(do_setrlimit, &c); if (c.err) { - errno = c.err; + if (c.err>0) errno = c.err; return -1; } return 0; -- cgit v1.2.1