posix_tty: free leak of xf86SetStrOption return value.
Pointed out by coverity. Signed-off-by: Dave Airlie <airlied@redhat.com>
This commit is contained in:
parent
b6c29a881e
commit
cad3a1a82d
|
@ -188,7 +188,7 @@ xf86SetSerial(int fd, XF86OptionPtr options)
|
|||
{
|
||||
struct termios t;
|
||||
int val;
|
||||
const char *s;
|
||||
char *s;
|
||||
int baud, r;
|
||||
|
||||
if (fd < 0)
|
||||
|
@ -264,8 +264,10 @@ xf86SetSerial(int fd, XF86OptionPtr options)
|
|||
}
|
||||
else {
|
||||
xf86Msg(X_ERROR, "Invalid Option Parity value: %s\n", s);
|
||||
free(s);
|
||||
return -1;
|
||||
}
|
||||
free(s);
|
||||
}
|
||||
|
||||
if ((val = xf86SetIntOption(options, "Vmin", -1)) != -1) {
|
||||
|
@ -291,8 +293,10 @@ xf86SetSerial(int fd, XF86OptionPtr options)
|
|||
}
|
||||
else {
|
||||
xf86Msg(X_ERROR, "Invalid Option FlowControl value: %s\n", s);
|
||||
free(s);
|
||||
return -1;
|
||||
}
|
||||
free(s);
|
||||
}
|
||||
|
||||
if ((xf86SetBoolOption(options, "ClearDTR", FALSE))) {
|
||||
|
|
Loading…
Reference in New Issue
Block a user