boot loader: Improve code of elf header initialization
authorJohn Marino <draco@marino.st>
Sat, 14 Nov 2015 13:26:11 +0000 (14:26 +0100)
committerJohn Marino <draco@marino.st>
Sat, 14 Nov 2015 13:26:11 +0000 (14:26 +0100)
This has been driving me crazy.  This change has no practical difference,
but I didn't like seeing the code loop through headers twice for no good
reason.  This moves all the initialization to a single loop.

sys/boot/common/load_elf_obj.c

index 91da636..d9b3886 100644 (file)
@@ -217,9 +217,8 @@ __elfN(obj_loadimage)(struct preloaded_file *fp, elf_file_t ef, u_int64_t off)
         * We store the load address as a non-zero sh_addr value.
         * Start with the code/data and bss.
         */
-       for (i = 0; i < hdr->e_shnum; i++)
-               shdr[i].sh_addr = 0;
        for (i = 0; i < hdr->e_shnum; i++) {
+               shdr[i].sh_addr = 0;
                if (shdr[i].sh_size == 0)
                        continue;
                switch (shdr[i].sh_type) {