Change in flashrom[1.0.x]: spi25_statusreg: Return defined value on failed RDSR

[Nico Huber (Code Review)]

Nico Huber has uploaded this change for review. ( https://review.coreboot.org/22349

Change subject: spi25_statusreg: Return defined value on failed RDSR
......................................................................

spi25_statusreg: Return defined value on failed RDSR

The interface of spi_read_status_register() is broken and can't return
errors. Let's not return random stack data at least.

Change-Id: I714b20001a5443bba665c2e0061ca14069777581
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/22017
Tested-by: build bot (Jenkins) 
Reviewed-by: David Hendricks 
(cherry picked from commit 1f081530b60ee805532f106f59cc33973e160481)
---
M spi25_statusreg.c
1 file changed, 4 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/49/22349/1

diff --git a/spi25_statusreg.c b/spi25_statusreg.c
index 01a6862..05c7acf 100644
--- a/spi25_statusreg.c
+++ b/spi25_statusreg.c
@@ -117,8 +117,11 @@

/* Read Status Register */
    ret = spi_send_command(flash, sizeof(cmd), sizeof(readarr), cmd, readarr);
-	if (ret)
+	if (ret) {
    	msg_cerr("RDSR failed!\n");
+		/* FIXME: We should propagate the error. */
+		return 0;
+	}

return readarr[0];
 }