diff options
| author | AlexChen <alex.chen@huawei.com> | 2020-11-02 18:55:52 +0800 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2021-09-15 14:42:48 +0200 |
| commit | fae0b0de71eb823dbe89f5784f3c3971485525cd (patch) | |
| tree | 1bced10ebd431b2a21d3559e6e048abae7b23c02 /util/qemu-openpty.c | |
| parent | 831aaf24967a49d7750090b9dcfd6bf356f16529 (diff) | |
| download | qemu-fae0b0de71eb823dbe89f5784f3c3971485525cd.zip qemu-fae0b0de71eb823dbe89f5784f3c3971485525cd.tar.gz qemu-fae0b0de71eb823dbe89f5784f3c3971485525cd.tar.bz2 | |
util: Remove redundant checks in the openpty()
As we can see from the following function call stack, amaster and aslave
can not be NULL: char_pty_open() -> qemu_openpty_raw() -> openpty().
In addition, according to the API specification for openpty():
https://www.gnu.org/software/libc/manual/html_node/Pseudo_002dTerminal-Pairs.html,
the arguments name, termp and winp can all be NULL, but arguments amaster or aslave
can not be NULL.
Finally, amaster and aslave has been dereferenced at the beginning of the openpty().
So the checks on amaster and aslave in the openpty() are redundant. Remove them.
Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Alex Chen <alex.chen@huawei.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <5F9FE5B8.1030803@huawei.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'util/qemu-openpty.c')
| -rw-r--r-- | util/qemu-openpty.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/util/qemu-openpty.c b/util/qemu-openpty.c index eb17f5b..427f43a 100644 --- a/util/qemu-openpty.c +++ b/util/qemu-openpty.c @@ -80,10 +80,9 @@ static int openpty(int *amaster, int *aslave, char *name, (termp != NULL && tcgetattr(sfd, termp) < 0)) goto err; - if (amaster) - *amaster = mfd; - if (aslave) - *aslave = sfd; + *amaster = mfd; + *aslave = sfd; + if (winp) ioctl(sfd, TIOCSWINSZ, winp); |