You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@guacamole.apache.org by "Changkun Ou (JIRA)" <ji...@apache.org> on 2018/08/09 04:58:00 UTC
[jira] [Created] (GUACAMOLE-608) Daemon process may encounter dead
loop
Changkun Ou created GUACAMOLE-608:
-------------------------------------
Summary: Daemon process may encounter dead loop
Key: GUACAMOLE-608
URL: https://issues.apache.org/jira/browse/GUACAMOLE-608
Project: Guacamole
Issue Type: Bug
Components: guacd
Affects Versions: 0.9.14
Reporter: Changkun Ou
`guacd` involves a `__write_all` function to write instruction to write as much as possible, see https://github.com/apache/guacamole-server/blob/7c191d7be0441a1cb64c90ab62d6535f3798eacb/src/guacd/connection.c#L67
However system call `write` may return 0 and set `errno`, which is not verified in the function.
A possible case: `write` keeps return 0 and nothing writes to buffer, therefore the daemon process encounters a dead loop, furthermore, it leads CPU rate up to 99%.
A possible fix is:
```
int written = write(fd, buffer, length);
if (written < 0)
return -1;
if (written == 0 && errno > 0)
return -1;
```
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)