[coreboot-gerrit] Change in coreboot[master]: tianocore: Add more patches

build bot (Jenkins) (Code Review) gerrit at coreboot.org
Sun Jun 17 11:09:33 CEST 2018


build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27137 )

Change subject: tianocore: Add more patches
......................................................................


Patch Set 1:

(37 comments)

https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch
File payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch:

https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@16
PS1, Line 16:    } else {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@17
PS1, Line 17:      Length = strlen(Str);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@18
PS1, Line 18:      if (Length < 4) {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@19
PS1, Line 19: -      strncpy (TemStr + 4 - Length, Str, Length);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@20
PS1, Line 20: +      memcpy (TemStr + 4 - Length, Str, Length);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@21
PS1, Line 21:      } else {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@22
PS1, Line 22: -      strncpy (TemStr, Str + Length - 4, 4);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@23
PS1, Line 23: +      memcpy (TemStr, Str + Length - 4, 4);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@24
PS1, Line 24:      }
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@25
PS1, Line 25:    
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@26
PS1, Line 26:      sscanf (
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@28
PS1, Line 28:    //
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@29
PS1, Line 29:    FitStartPtr = (FIT_TABLE *) RelativeAddress;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@30
PS1, Line 30:  
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@31
PS1, Line 31: -  strncpy ((CHAR8 *) &FitStartPtr->CompAddress, FIT_SIGNATURE, 8);  // "_FIT_   "
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@32
PS1, Line 32: +  memcpy ((CHAR8 *) &FitStartPtr->CompAddress, FIT_SIGNATURE, 8);  // "_FIT_   "
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@33
PS1, Line 33:    assert (((VtfInfo->CompSize & 0x00FFFFFF) % 16) == 0);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@34
PS1, Line 34:    FitStartPtr->CompSize     = (VtfInfo->CompSize & 0x00FFFFFF) / 16;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@35
PS1, Line 35:    FitStartPtr->CompVersion  = MAKE_VERSION (VtfInfo->MajorVer, VtfInfo->MinorVer);
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/08_BaseTools-Fix-building-with-Werror-stringop-truncati.patch@36
PS1, Line 36: -- 
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch
File payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch:

https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@17
PS1, Line 17:    UINT8       BlockType;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@18
PS1, Line 18:    EFI_HII_STRING_PACKAGE_HDR *PkgHeader;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@19
PS1, Line 19:    
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@20
PS1, Line 20: -  if (mStringFileName == '\0' ) {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@21
PS1, Line 21: +  if (mStringFileName == NULL ) {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@22
PS1, Line 22:      return NULL;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@23
PS1, Line 23:    }
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@24
PS1, Line 24:  
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@30
PS1, Line 30:    
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@31
PS1, Line 31:    StringPtr = ConfigAltResp;
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@32
PS1, Line 32:    
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@33
PS1, Line 33: -  while (StringPtr != L'\0') {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@34
PS1, Line 34: +  while (StringPtr != NULL) {
DOS line endings


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@35
PS1, Line 35:      //
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@36
PS1, Line 36:      // 1. Find <ConfigHdr> GUID=...&NAME=...&PATH=...
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@37
PS1, Line 37:      //
trailing whitespace


https://review.coreboot.org/#/c/27137/1/payloads/external/tianocore/patches/09_gcc7-Fix-building-with-fpermissive.patch@38
PS1, Line 38: -- 
trailing whitespace



-- 
To view, visit https://review.coreboot.org/27137
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7e2efea930b27749b1097607ab143ce2f91e79ec
Gerrit-Change-Number: 27137
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <siro at das-labor.org>
Gerrit-CC: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Sun, 17 Jun 2018 09:09:33 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180617/a2d2a9e1/attachment.html>


More information about the coreboot-gerrit mailing list