discover/pxe: check for a valid boot option before adding

If we didn't find any valid boot options in the pxe buffer, we'll call
discover_context_add_boot_option with a NULL boot option.

This change adds a check before we try to add the boot option, and a
test to verify this situation.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

2 files changed, 20 insertions, 0 deletions

diff --git a/test/parser/Makefile.am b/test/parser/Makefile.am
index 9029544..e221bf2 100644
--- a/test/parser/Makefile.am
+++ b/test/parser/Makefile.am
@@ -41,6 +41,7 @@ TESTS = \
 	test-yaboot-device-override \
 	test-yaboot-default \
 	test-yaboot-rh8-ppc64 \
+	test-pxe-empty \
 	test-pxe-single \
 	test-pxe-initrd-in-append
 
diff --git a/test/parser/test-pxe-empty.c b/test/parser/test-pxe-empty.c
new file mode 100644
index 0000000..eb3b758
--- /dev/null
+++ b/test/parser/test-pxe-empty.c
@@ -0,0 +1,19 @@
+
+#include "parser-test.h"
+
+#if 0 /* PARSER_EMBEDDED_CONFIG */
+
+#endif
+
+void run_test(struct parser_test *test)
+{
+	struct discover_context *ctx;
+
+	test_read_conf_embedded(test);
+	test_set_conf_source(test, "tftp://host/dir/conf.txt");
+	test_run_parser(test, "pxe");
+
+	ctx = test->ctx;
+
+	check_boot_option_count(ctx, 0);
+}