linux-kernel - RE: [PATCH] cpumask: Omit terminating null byte in cpumap_print_{list,bitmask}_to

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aa4bc59c44b345ae814c61f6593a7178@hisilicon.com>
Date:   Thu, 16 Sep 2021 22:53:39 +0000
From:   "Song Bao Hua (Barry Song)" <song.bao.hua@...ilicon.com>
To:     Tobias Klauser <tklauser@...tanz.ch>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Jonathan Cameron <jonathan.cameron@...wei.com>,
        "tiantao (H)" <tiantao6@...ilicon.com>
CC:     Andrew Morton <akpm@...ux-foundation.org>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Yury Norov <yury.norov@...il.com>,
        "Peter Zijlstra" <peterz@...radead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] cpumask: Omit terminating null byte in
 cpumap_print_{list,bitmask}_to_buf



> -----Original Message-----
> From: Tobias Klauser [mailto:tklauser@...tanz.ch]
> Sent: Friday, September 17, 2021 10:27 AM
> To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>; Jonathan Cameron
> <jonathan.cameron@...wei.com>; tiantao (H) <tiantao6@...ilicon.com>; Song Bao
> Hua (Barry Song) <song.bao.hua@...ilicon.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>; Andy Shevchenko
> <andriy.shevchenko@...ux.intel.com>; Yury Norov <yury.norov@...il.com>; Peter
> Zijlstra <peterz@...radead.org>; linux-kernel@...r.kernel.org
> Subject: [PATCH] cpumask: Omit terminating null byte in
> cpumap_print_{list,bitmask}_to_buf
> 
> The changes in the patch series [1] introduced a terminating null byte
> when reading from cpulist or cpumap sysfs files, for example:
> 
>   $ xxd /sys/devices/system/node/node0/cpulist
>   00000000: 302d 310a 00                             0-1..
> 
> Before this change, the output looked as follows:
> 
>   $ xxd /sys/devices/system/node/node0/cpulist
>   00000000: 302d 310a                                0-1.

If we don't use xxd, I don't see any actual harm of this NULL byte
by cat, lscpu, numactl etc. this doesn't break them at all.

if we only want to make sure the output is exactly same with before
for every single character, this patch is right.

> 
> Fix this regression by excluding the terminating null byte from the
> returned length in cpumap_print_list_to_buf and
> cpumap_print_bitmask_to_buf.
> 
> [1]
> https://lore.kernel.org/all/20210806110251.560-1-song.bao.hua@hisilicon.co
> m/
> 
> Fixes: 1fae562983ca ("cpumask: introduce cpumap_print_list/bitmask_to_buf to
> support large bitmask and list")
> Signed-off-by: Tobias Klauser <tklauser@...tanz.ch>
> ---
>  include/linux/cpumask.h | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h
> index 5d4d07a9e1ed..1e7399fc69c0 100644
> --- a/include/linux/cpumask.h
> +++ b/include/linux/cpumask.h
> @@ -996,14 +996,15 @@ cpumap_print_to_pagebuf(bool list, char *buf, const struct
> cpumask *mask)
>   * cpumask; Typically used by bin_attribute to export cpumask bitmask
>   * ABI.
>   *
> - * Returns the length of how many bytes have been copied.
> + * Returns the length of how many bytes have been copied, excluding
> + * terminating '\0'.
>   */
>  static inline ssize_t
>  cpumap_print_bitmask_to_buf(char *buf, const struct cpumask *mask,
>  		loff_t off, size_t count)
>  {
>  	return bitmap_print_bitmask_to_buf(buf, cpumask_bits(mask),
> -				   nr_cpu_ids, off, count);
> +				   nr_cpu_ids, off, count) - 1;
>  }
> 
>  /**
> @@ -1018,7 +1019,7 @@ cpumap_print_list_to_buf(char *buf, const struct cpumask
> *mask,
>  		loff_t off, size_t count)
>  {
>  	return bitmap_print_list_to_buf(buf, cpumask_bits(mask),
> -				   nr_cpu_ids, off, count);
> +				   nr_cpu_ids, off, count) - 1;
>  }
> 
>  #if NR_CPUS <= BITS_PER_LONG
> --
> 2.33.0

Thanks
Barry