build-sys: add syslinux test files to tarball

Add test for syslinux converter

Don't remove initrd= parameter.

Based on simplified patch by Lunar.

Reported by: Lunar

syslinux_parse: Always output comments even if no entries are found.

diskfilter_make_raid: more memory leaks in failure path

disk/lvm: Use zalloc to ensure that segments are initialised to sane value.

Reported by: EmanueL Czirai.

multiboot2: Fix information request tag size calculation

Signed-off-by: Daniel Kiper <daniel.kiper@oracle.com>

diskfilter: fix double free of lv names for mdraid

Avoid micro-optimization in grub_diskfilter_make_raid and make sure
name and fullname are independent strings. This avoids need to special
case it everywhere else.

Also fix memory leak in failure case in grub_diskfilter_make_raid.

Closes: 41582

diskfilter: fix crash in validate_lv for mdraid arrays

Commit 750f4bacd3262376ced3f837d8dc78f834ca233a put LV validation before
actual vg assignment. Make grub_diskfilter_make_raid to assign ->vg as
happens in other cases for consistency. Also clean up redundant code and add
explicit NULL lv->vg check in validate_lv.

Also fix segment validation in validate_lv; it became obvious when crash
was fixed.

Closes: 44199

util: mkimage, fix gcc5 build failure

gcc5 reports:
../util/mkimage.c: In function 'grub_install_get_image_target':
../util/mkimage.c:954:5: error: loop exit may only be reached after undefined behavior [-Werror=aggressive-loop-optimizations]
     && j < ARRAY_SIZE (image_targets[i].names); j++)
     ^
../util/mkimage.c:953:39: note: possible undefined statement is here
      for (j = 0; image_targets[i].names[j]
                                        ^

Well, let's move the index 'j' test before accesing the array to:
1) make the loop obvious
2) make gcc happy

Signed-off-by: Jiri Slaby <jslaby@suse.cz>

arm: implement additional relocations generated by gcc 4.9 at -O3

GCC 4.9 also generates R_ARM_THM_MOVW_ABS_NC and R_ARM_THM_MOVT_ABS,
as an alternative to ABS32.

Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>

setup: fix blocklist size calculation

Found by: Coverity scan.

grub-fstest: fix descriptor leak

Found by: Coverity scan.

net/pxe: fix error condition

Test return value of grub_netbuff_reserve(), buf itself cannot be
NULL here.

Found by: Coverity scan.

grub-mkimage: fix potential NULL pointer dereference

Move fatal check whether symtab_section is NULL before first reference.

Found by: Coverity scan.

net/ip: check result of grub_netbuff_push

Found by: Coverity scan.

tests: add test command file tests

This requires access to files in both host and grub image, so
implementing as separate test unit instead of script test was
more easy.

test: consistently use TMPDIR and same name pattern for temp files

test: fix previous commit - we need to return from subexpression

( ... ) was processed recursively, we need to return from it. Revert
this change.

test: do not stop after first file test or closing bracket

Closes: 44115

configure.ac: don't use -msoft-float for arm64

aarch64 toolchains do not support the -msoft-float option added by
commit 3661261f. Insted, for arm64 use -march=armv8-a+nofp+nosimd.

Reported-by: Ryan Harkin <ryan.harkin@linaro.org>

script/execute.c: fix memory leak.

Make sure to continue loop over array after failure to free
allocated strings.

Found by: Coverity scan.

syslinux_parse: fix memory leak.

Found by: Coverity scan.

Change quotes to match overall style in NEWS

loader/xnu: fix memory leak.

Foound by: Coverity scan.

util/grub-probe: fix memory leaks.

Found by: Coverity scan.

fs/hfsplus: fix memory leak.

Found by: Coverity scan.

fs/zfs/zfscrypt.c: fix indentation.

fs/zfs/zfscrypt.c: fix memory leaks.

Found by: Coverity scan.

commands/parttool: fix memory leak.

Found by: Coverity scan.

fs/zfs/zfs.c: fix memory leak.

Found by: Coverity scan.

linux/ofpath: fix descriptor leak

Found by: Coverity scan

linux/hostdisk: use strncpy instead of strlcpy

strlcpy is not available on Linux as part of standard libraries.
It probably is not worth extra configure checks espicially as we
need to handle missing function anyway.

Document intentional fallthroughs.

Found by: Coverity scan.

linux/ofpath: Fix error handling.

Found by: Coverity Scan.

linux/hostdisk: Limit strcpy size to buffer size.

Found by: Coverity scan.

fs/zfscrypt: Add missing explicit cast.

Found by: Coverity scan.

fs/zfs: Fix error handling.

Found by: Coverity Scan.

fs/{cbfs,cpio}: Remove useless check if mode is NULL.

Callers already ensure that it's not null.

Found by: Coverity Scan.

commands/acpi: Use ALIGN_UP rather than manual expression.

Improves readability and hopefully automatic scanning.

Found by: Coverity Scan.

util/setup: fix memory leak.

Found by: Coverity scan.

util/mkimage: fix memory leaks.

Found by: Coverity scan.

util/grub-mount: fix descriptor leak.

Found by: Coverity scan.

util/grub-mkstandalone: fix memory leak.

Found by: Coverity scan.

util/grub-install: rearrange code to avoid memory leak.

Found by: Coverity scan.

linux/getroot: fix memory leak.

Found by: Coverity scan.

util/install: fix memory leak.

Found by: Coverity scan.

util/setup: fix memory leak.

Found by: Coverity scan.

linux/ofpath: fix various memory leaks.

Found by: Coverity scan.

linux/getroot: fix descriptor leak.

Found by: Coverity scan.

util/misc.c: Check ftello return value.

Found by: Coverity scan.

grub-macbless: Fix resource leak.

Found by: Coverity scan.

grub-install: Fix memory leak.

Found by: Coverity scan.

grub-install-common: Fix sizeof usage.

Found by: Coverity scan.

util/getroot: Add missing grub_disk_close.

Found by: Coverity scan.

vbe: Fix incorrect register usage.

Found by: Coverity scan.

unix/password: Fix file descriptor leak.

Found by: Coverity scan.

linux/getroot: Fix error handling.

Found by: Coverity scan.

linux/blocklist: Fix memory leak.

Found by: Coverity scan.

devmapper/getroot: Fix memory leak.

Found by: Coverity scan.

normal/misc: Close device on all pathes.

Found by: Coverity scan.

normal/main: Fix error handling.

Found by: Coverity scan.

xnu: Add missing error check.

Found by: Coveriy scan.

plan9: Add missing grub_device_close.

Found by: Coverity scan.

multiboot: Simplify to avoid confusing assignment.

Found by: Coverity scan.

bsd: Add missing null-pointer check.

Found by: Coverity scan.

lib/syslinux_parse: Add missing error check.

Found by: Coverity scan.

lib/syslinux_parse: Fix memory leak.

Found by: Coveriy scan.

lib/syslinux_parse: Add missing alloc check.

Found by: Coverity scan.

i386/pc/mmap: Fix memset size.

Found by: Coverity scan.

gfxmenu/theme_loader: Add missing allos error check.

Found by: Coverity scan.

gfxmenu/icon_manager: Fix null pointer dereference.

Found by: Coverity scan.

fs/ufs: Add missing error check.

Found by: Coverity scan.

configure.ac: Always add -D_FILE_OFFSET_BITS=64.

fs/sfs: Fix error check and add sanity check.

Found by: Coverity scan.

fs/reiserfs: Fix sector count overflow.

Found by: Coverity scan.

fs/ntfs: Add sizes sanity checks.

Found by: Coverity scan.

fs/ntfs: Add missing free.

Found by: Coverity scan.

fs/minix: Fix sector promotion to 64-bit.

While on it make GRUB_MINIX_ZONE2SECT into function.

Found by: Coverity scan

grub_iso9660_read: Explicitly check read_node return value.

Not really needed as grub_errno is already checked but is nicer.

Found by: Coverity scan.

commands/fileXX: Fix remaining memory leak.

Found by: Coverity Scan.

fs/hfs: Add pointer sanity checks.

Found by: Coverity scan.

fs/hfs/hfs_open: Check that mount succeeded.

Found by: Coverity scan.

fs/fat: Fix codepath to properly free on error.

Found by: Coverity scan.

fs/cpio_common: Add a sanity check on namesize.

Found by: Coverity scan.

fs/cbfs: Add missing free.

Found by: Coverity scan.

font: Add missing free.

Found by: Coverity Scan.

biosdisk: Add missing cast.

Found by: Coverity scan.

disk/geli: Add missing free.

Found by: Coverity scan.

disk/geli: Add missing seek success check.

Found by: Coverity scan.

disk/diskfilter: Add missing lv presence check.

Found by: Coverity scan.

disk/cryptodisk: Add missing error check.

Found by: Coverity scan.

disk/ahci: Fix device_map_range argument.

Argument is not used on x86, hence it's gone unnoticed.

Found by: Coverity scan.

disk/AFsplitter: check argument validity before doing any allocs.

This avoids possible memory leaks.

Found by: Coverity scan.

commands/wildcard: Add missing free.

Found by: Coverity scan.

commands/verify: Fix sha1 context zeroing-out.

Current code doesn't zero-out context completely. It's a minor issue
really as sha1 init already takes care of initing the context.

commands/tr: Simplify and fix missing parameter test.

Found by: Coverity scan

commands/syslinux: Add missing free.

Found by: Coverity scan.

commands/parttool: Add missing device close.

Found by: Coverity scan.

commands/nativedisk: Add missing device_close.

Found by: Coverity scan.