diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-10-11 19:33:13 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-10-13 17:56:54 -0400 |
| commit | 7ff7394d9e4e9189c30fd018235e6b1728c6f2d0 (patch) | |
| tree | 37a98cdb42d8b62b56aeb6fad7fd00299449d38d /src/shared/efivars.c | |
| parent | fb1316462952d17d6ebf19c3f093b730c13016a7 (diff) | |
Never call qsort on potentially NULL arrays
This extends 62678ded 'efi: never call qsort on potentially
NULL arrays' to all other places where qsort is used and it
is not obvious that the count is non-zero.
Diffstat (limited to 'src/shared/efivars.c')
| -rw-r--r-- | src/shared/efivars.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/shared/efivars.c b/src/shared/efivars.c index c015b16857..f3eb6a6e5d 100644 --- a/src/shared/efivars.c +++ b/src/shared/efivars.c @@ -384,8 +384,7 @@ int efi_get_boot_options(uint16_t **options) { list[count ++] = id; } - if (list) - qsort(list, count, sizeof(uint16_t), cmp_uint16); + qsort_safe(list, count, sizeof(uint16_t), cmp_uint16); *options = list; return count; |