[coreboot-gerrit] Change in em100[master]: Add network update functionality

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/em100/+/37072 )

Change subject: Add network update functionality
......................................................................

Patch Set 14: Code-Review+1

(5 comments)

https://review.coreboot.org/c/em100/+/37072/14/curl.c 
File curl.c:

https://review.coreboot.org/c/em100/+/37072/14/curl.c@51 
PS14, Line 51: =
maybe add space around the equals symbol here and on the next line?

also on `pos+1` below

https://review.coreboot.org/c/em100/+/37072/14/curl.c@62 
PS14, Line 62: written
Couldn't this be returned directly?

https://review.coreboot.org/c/em100/+/37072/14/curl.c@83 
PS14, Line 83: 		return -1;
Potential leak?

fclose(file);

https://review.coreboot.org/c/em100/+/37072/14/curl.c@145 
PS14, Line 145: 		return 1;
Potential leak?

fclose(old);
 fclose(new);	
 unlink(tmp_version);

https://review.coreboot.org/c/em100/+/37072/14/curl.c@153 
PS14, Line 153: !=2
Missing space here and on the next line