d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

pututxline: Don't set errno=0 in subfunctions.

Browse Source 
The functions utx_active_add(), utx_active_remove(), utx_lastlogin_add() and
utx_log_add() set errno to 0 if they are successful. This not only violates
POSIX if pututxline() is successful, but may also overwrite a valid error
with 0 if, for example, utx_lastlogin_add() fails while utx_log_add()
succeeds.

Reviewed by:	ed
This commit is contained in:
jilles 2013-04-17 21:08:15 +00:00
parent fc3fc9c036
commit f83f35001e
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/libc/gen/pututxline.c
View File

@ -131,7 +131,8 @@ exact:
else
error = 0;
fclose(fp);
errno = error;
if (error != 0)
errno = error;
return (error == 0 ? 0 : 1);
}
@ -169,7 +170,8 @@ utx_active_remove(struct futx *fu)
}
fclose(fp);
errno = error;
if (ret != 0)
errno = error;
return (ret);
}
@ -225,7 +227,8 @@ utx_lastlogin_add(const struct futx *fu)
ret = -1;
}
fclose(fp);
errno = error;
if (ret == -1)
errno = error;
return (ret);
}
@ -277,7 +280,8 @@ utx_log_add(const struct futx *fu)
else
error = 0;
_close(fd);
errno = error;
if (error != 0)
errno = error;
return (error == 0 ? 0 : 1);
}