[flashrom] [PATCH] Fix out-of-bounds access if all erase functions fail

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Apr 15 07:31:29 CEST 2011


Fix out-of-bounds access if all erase functions fail.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

Index: flashrom-fix_erasefunctions_nullpointer/flashrom.c
===================================================================
--- flashrom-fix_erasefunctions_nullpointer/flashrom.c	(Revision 1280)
+++ flashrom-fix_erasefunctions_nullpointer/flashrom.c	(Arbeitskopie)
@@ -1514,8 +1514,12 @@
  	memcpy(curcontents, oldcontents, size);

  	for (k = 0; k<  NUM_ERASEFUNCTIONS; k++) {
+		if (!usable_erasefunctions) {
+			msg_cdbg("No usable erase functions left.\n");
+			break;
+		}
  		msg_cdbg("Looking at blockwise erase function %i... ", k);
-		if (check_block_eraser(flash, k, 1)&&  usable_erasefunctions) {
+		if (check_block_eraser(flash, k, 1)) {
  			msg_cdbg("Looking for another erase function.\n");
  			continue;
  		}


-- 
http://www.hailfinger.org/





More information about the flashrom mailing list