[coreboot-gerrit] New patch to review for coreboot: lib/edid.c: Return parsed string if it has a valid termination

Arthur Heymans (arthur@aheymans.xyz) gerrit at coreboot.org
Thu Nov 24 08:39:15 CET 2016


Arthur Heymans (arthur at aheymans.xyz) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17592

-gerrit

commit abae732507c46eb338af2341ff55a6a6a9aecf6f
Author: Arthur Heymans <arthur at aheymans.xyz>
Date:   Thu Nov 24 08:19:57 2016 +0100

    lib/edid.c: Return parsed string if it has a valid termination
    
    Change-Id: I9f80b470a3d889bb15219e782173888b4b39e5eb
    Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
 src/lib/edid.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/lib/edid.c b/src/lib/edid.c
index d1b9c9f..7ec71cf 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -186,7 +186,8 @@ extract_string(unsigned char *x, int *valid_termination, int len)
 			if (x[i] != 0x20) {
 				*valid_termination = 0;
 				return ret;
-			}
+			} else
+				return ret;
 		} else if (x[i] == 0x0a) {
 			seen_newline = 1;
 		} else {



More information about the coreboot-gerrit mailing list