RE: [PATCH] serial: samsung: protect NULL dereference of clock name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Kyoungil Kim wrote:
> 
> From: KeyYoung Park <keyyoung.park@xxxxxxxxxxx>
> 
> When priting the serial clock source, if clock source name is null,
> kernel reference NULL point.
> 
> Signed-off-by: KeyYoung Park <keyyoung.park@xxxxxxxxxxx>
> Signed-off-by: Huisung Kang <hs1218.kang@xxxxxxxxxxx>
> Signed-off-by: Kyoungil Kim <ki0351.kim@xxxxxxxxxxx>

(Cc'ed Greg)

Acked-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>

Greg, could you please pick this up?

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

> ---
>  drivers/tty/serial/samsung.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
> index ea76b40..77484e0 100644
> --- a/drivers/tty/serial/samsung.c
> +++ b/drivers/tty/serial/samsung.c
> @@ -1160,7 +1160,8 @@ static ssize_t s3c24xx_serial_show_clksrc(struct
> device *dev,
>  	struct uart_port *port = s3c24xx_dev_to_port(dev);
>  	struct s3c24xx_uart_port *ourport = to_ourport(port);
> 
> -	return snprintf(buf, PAGE_SIZE, "* %s\n", ourport->baudclk->name);
> +	return snprintf(buf, PAGE_SIZE, "* %s\n",
> +			ourport->baudclk->name ?: "(null)");
>  }
> 
>  static DEVICE_ATTR(clock_source, S_IRUGO, s3c24xx_serial_show_clksrc,
> NULL);
> --
> 1.7.1


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux