[PATCH v2] part: Add check for NULL dev_part_str

Simon Glass sjg at chromium.org
Wed May 19 17:34:08 CEST 2021


On Sat, 15 May 2021 at 12:14, Sean Anderson <seanga2 at gmail.com> wrote:
>
> Some callers (e.g. cmd/fs.c) of fs_set_blk_dev may use a NULL dev_part_str.
> While blk_get_device_part_str handles this fine,
> part_get_info_by_dev_and_name does not. This fixes commands crashing when
> implicitly using bootdevice.
>
> The unit test has also been updated to set bootdevice to a known value and
> to restore it after we are done.
>
> Fixes: 7194527b6a ("cmd: fs: Use part_get_info_by_dev_and_name_or_num to parse partitions")
> Reported-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Signed-off-by: Sean Anderson <seanga2 at gmail.com>
> ---
>
> Changes in v2:
> - Fix unit test failing if bootdevice was set
>
>  disk/part.c    |  6 +++++-
>  test/dm/part.c | 39 +++++++++++++++++++++++++++++++--------
>  2 files changed, 36 insertions(+), 9 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list