Harshit Sharma has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT]lib: Introduce memory bugs to help test ASan ......................................................................
[DONOTSUBMIT]lib: Introduce memory bugs to help test ASan
Introduces a few memory bugs into cbfs code to make testing easier for the ones who are willing to try out ASan on their hardware.
Change-Id: I0839f2fd2863934ec28e2322bab04b9cc33363b4 Signed-off-by: Harshit Sharma harshitsharmajs@gmail.com --- M src/lib/cbfs.c 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/43938/1
diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c index cb66f81..839de94 100644 --- a/src/lib/cbfs.c +++ b/src/lib/cbfs.c @@ -330,9 +330,30 @@ return 0; }
+#if ENV_RAMSTAGE + int global_array[5] = {-1}; +#endif + int cbfs_boot_region_device(struct region_device *rdev) { + int stack_array[5], i, *p; boot_device_init(); + + /* global out-of-bounds */ +#if ENV_RAMSTAGE + for (i = 10; i > 0; i--) + global_array[i] = i; +#endif + /* stack out-of-bounds */ + for (i = 10; i > 0; i--) + stack_array[i] = i; + + /* use-after-scope */ + { + int x = 5; + p = &x; + } + *p = 10; return vboot_locate_cbfs(rdev) && fmap_locate_area_as_rdev("COREBOOT", rdev); }
Harshit Sharma has uploaded a new patch set (#2). ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
[DONOTSUBMIT] lib: Introduce memory bugs to help test ASan
Introduces a few memory bugs into cbfs code to make testing easier for the ones who are willing to try out ASan on their hardware.
Change-Id: I0839f2fd2863934ec28e2322bab04b9cc33363b4 Signed-off-by: Harshit Sharma harshitsharmajs@gmail.com --- M src/lib/cbfs.c 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/43938/2
Hello build bot (Jenkins), Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/43938
to look at the new patch set (#3).
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
[DONOTSUBMIT] lib: Introduce memory bugs to help test ASan
Introduces a few memory bugs into cbfs code to make testing easier for the ones who are willing to try out ASan on their hardware.
Also, enable ASAN_IN_ROMSTAGE by default on all x86 platforms.
Change-Id: I0839f2fd2863934ec28e2322bab04b9cc33363b4 Signed-off-by: Harshit Sharma harshitsharmajs@gmail.com --- M src/Kconfig M src/lib/cbfs.c 2 files changed, 22 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/43938/3
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3:
j
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: Code-Review+1
uwe taz has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Patch Set 3: -Code-Review
Werner Zeh has removed Name of user not set #1003096 from this change. ( https://review.coreboot.org/c/coreboot/+/43938 )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Removed reviewer null with the following votes:
* Verified+1 by Name of user not set (1003096)
Stefan Reinauer has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/43938?usp=email )
Change subject: [DONOTSUBMIT] lib: Introduce memory bugs to help test ASan ......................................................................
Abandoned