fakeidentd: avoid extra fcntl calls
This commit is contained in:
parent
19250813a8
commit
c14c95ec89
@ -41,11 +41,13 @@ static int do_rd(int fd, void **paramp)
|
|||||||
{
|
{
|
||||||
identd_buf_t *buf = *paramp;
|
identd_buf_t *buf = *paramp;
|
||||||
char *cur, *p;
|
char *cur, *p;
|
||||||
|
int retval = 0; /* session is ok (so far) */
|
||||||
int sz;
|
int sz;
|
||||||
|
|
||||||
cur = buf->buf + buf->pos;
|
cur = buf->buf + buf->pos;
|
||||||
|
|
||||||
fcntl(fd, F_SETFL, buf->fd_flag);
|
if (buf->fd_flag & O_NONBLOCK)
|
||||||
|
fcntl(fd, F_SETFL, buf->fd_flag);
|
||||||
sz = safe_read(fd, cur, sizeof(buf->buf) - buf->pos);
|
sz = safe_read(fd, cur, sizeof(buf->buf) - buf->pos);
|
||||||
|
|
||||||
if (sz < 0) {
|
if (sz < 0) {
|
||||||
@ -59,18 +61,18 @@ static int do_rd(int fd, void **paramp)
|
|||||||
p = strpbrk(cur, "\r\n");
|
p = strpbrk(cur, "\r\n");
|
||||||
if (p)
|
if (p)
|
||||||
*p = '\0';
|
*p = '\0';
|
||||||
if (p || !sz || buf->pos == sizeof(buf->buf)) {
|
if (!p && sz && buf->pos <= sizeof(buf->buf))
|
||||||
/* fd is still in nonblocking mode - we never block here */
|
goto ok;
|
||||||
fdprintf(fd, "%s : USERID : UNIX : %s\r\n", buf->buf, bogouser);
|
/* Terminate session. If we are in server mode, then
|
||||||
goto term;
|
* fd is still in nonblocking mode - we never block here */
|
||||||
}
|
fdprintf(fd, "%s : USERID : UNIX : %s\r\n", buf->buf, bogouser);
|
||||||
ok:
|
|
||||||
fcntl(fd, F_SETFL, buf->fd_flag & ~O_NONBLOCK);
|
|
||||||
return 0;
|
|
||||||
term:
|
term:
|
||||||
fcntl(fd, F_SETFL, buf->fd_flag & ~O_NONBLOCK);
|
|
||||||
free(buf);
|
free(buf);
|
||||||
return 1;
|
retval = 1; /* terminate */
|
||||||
|
ok:
|
||||||
|
if (buf->fd_flag & O_NONBLOCK)
|
||||||
|
fcntl(fd, F_SETFL, buf->fd_flag & ~O_NONBLOCK);
|
||||||
|
return retval;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int do_timeout(void **paramp)
|
static int do_timeout(void **paramp)
|
||||||
|
Loading…
Reference in New Issue
Block a user