diff options
author | Jianfeng Tan <henry.tjf@antfin.com> | 2019-09-20 17:49:18 +0000 |
---|---|---|
committer | Jianfeng Tan <henry.tjf@antfin.com> | 2019-09-20 17:57:25 +0000 |
commit | 223481e92743f305ada22689f4ba41b36119d5fc (patch) | |
tree | 61a74b246790d3e06db7b158405a347b5a2eacf4 /pkg/sentry/mm | |
parent | e9af227a61b836310fdd0c8543c31afe094af5ae (diff) |
fix set hostname
Previously, when we set hostname:
$ strace hostname abc
...
sethostname("abc", 3) = -1 ENAMETOOLONG (File name too long)
...
According to man 2 sethostname:
"The len argument specifies the number of bytes in name. (Thus, name
does not require a terminating null byte.)"
We wrongly use the CopyStringIn() to check terminating zero byte in
the implementation of sethostname syscall.
To fix this, we use CopyInBytes() instead.
Fixes: #861
Reported-by: chenglang.hy <chenglang.hy@antfin.com>
Signed-off-by: Jianfeng Tan <henry.tjf@antfin.com>
Diffstat (limited to 'pkg/sentry/mm')
0 files changed, 0 insertions, 0 deletions