[Xen-devel] [ovmf baseline-only test] 72127: all pass

[Platform Team regression test user <citrix-osstest@xxxxxxxxxxxxxx>]

This run is configured for baseline tests only.

flight 72127 ovmf real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/72127/

Perfect :-)
All tests in this flight passed as required
version targeted for testing:
 ovmf                 a3a4737051010a94832f7bceaa1fa414d7259da0
baseline version:
 ovmf                 91cc526b15ffbbbdec5a57906596f37e059f80be

Last test of basis    72125  2017-09-19 08:19:58 Z    0 days
Testing same since    72127  2017-09-19 16:20:35 Z    0 days    1 attempts

------------------------------------------------------------
People who touched revisions under test:
  Hao Wu <hao.a.wu@xxxxxxxxx>
  Yonghong Zhu <yonghong.zhu@xxxxxxxxx>

jobs:
 build-amd64-xsm                                              pass    
 build-i386-xsm                                               pass    
 build-amd64                                                  pass    
 build-i386                                                   pass    
 build-amd64-libvirt                                          pass    
 build-i386-libvirt                                           pass    
 build-amd64-pvops                                            pass    
 build-i386-pvops                                             pass    
 test-amd64-amd64-xl-qemuu-ovmf-amd64                         pass    
 test-amd64-i386-xl-qemuu-ovmf-amd64                          pass    


------------------------------------------------------------
sg-report-flight on osstest.xs.citrite.net
logs: /home/osstest/logs
images: /home/osstest/images

Logs, config files, etc. are available at
    http://osstest.xs.citrite.net/~osstest/testlogs/logs

Test harness code can be found at
    http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary


Push not applicable.

------------------------------------------------------------
commit a3a4737051010a94832f7bceaa1fa414d7259da0
Author: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
Date:   Fri Sep 15 16:14:17 2017 +0800

    BaseTools: Fix a bug to correct SourceFileList
    
    We met a case that use two microcode files in the Microcode.inf file,
    one is .mcb file, another is .txt file. then it cause build failure
    because the SourceFileList include the .txt file's output file, while
    this output file is still not be generated, so it cause
    GetFileDependency report failure.
    
    Cc: Liming Gao <liming.gao@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Yonghong Zhu <yonghong.zhu@xxxxxxxxx>
    Reviewed-by: Liming Gao <liming.gao@xxxxxxxxx>

commit 880ec68338541b63672ea521c0dffee181df8ede
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Fri Sep 15 08:57:40 2017 +0800

    MdeModulePkg/UdfDxe: Refine enum member naming style
    
    Similar to the naming style for variables, it's better for the name of
    members in a enum type to avoid using only upper-case letters.
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 3f92b104930ba582924da578a12ee0062881ab7b
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Fri Sep 15 08:43:22 2017 +0800

    MdeModulePkg/Udf: Avoid declaring and initializing local GUID variable
    
    The local GUID variable 'UdfDevPathGuid', it has been initialized during
    its declaration.
    
    For better coding style, this commit uses a global variable instead.
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 32492fee2d7e7116b5970266915285d043e0c0e1
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Sep 14 16:02:42 2017 +0800

    MdeModulePkg/UdfDxe: Avoid short (single character) variable name
    
    In ResolveSymlink(), replace the following variable:
    CHAR16              *C;
    
    with:
    CHAR16              *Char;
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 077f8c4372cc68efea91243dd1fe77d41315444d
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Sep 14 13:19:13 2017 +0800

    MdeModulePkg/Udf: Refine function description comments
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 12b83f5664a437106a9230d722e43558034aa61e
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Sep 14 10:52:47 2017 +0800

    MdeModulePkg/UdfDxe: Use compare operator for non-boolean comparisons
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Cc: Dandan Bi <dandan.bi@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit fedec0a38ef252ad838fe6d6db3cc4d28c7d42dd
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Sep 14 10:44:59 2017 +0800

    MdeModulePkg/UdfDxe: Fix operands of different size in bitwise OP
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

commit 3fa40d588af9cac947fd2abac652ebb95411786c
Author: Hao Wu <hao.a.wu@xxxxxxxxx>
Date:   Thu Sep 14 10:15:53 2017 +0800

    MdeModulePkg/UdfDxe: Add checks to ensure no possible NULL ptr deref
    
    Case 1 - Within DuplicateFid() & DuplicateFe():
    The call to AllocateCopyPool() may return NULL.
    Add ASSERTs as checks.
    
    Case 2 - Within UdfRead():
    Add ASSERT to ensure 'NewFileEntryData' returned from FindFileEntry()
    will not be NULL pointer.
    
    Case 3 - Within GetAllocationDescriptorLsn():
    The return value of 'GetPdFromLongAd (Volume, ParentIcb)' may be NULL,
    and it will be passed into function GetShortAdLsn() which will
    dereference it.
    Add ASSERT in GetShortAdLsn() as check.
    
    Case 4 - Within ReadFile():
    Add ASSERT to ensure 'Data' returned from GetAedAdsData() will not be NULL
    pointer.
    
    Case 5 - Within InternalFindFile():
    If both 'Parent->FileIdentifierDesc' and 'Icb' are NULL, then possible
    NULL pointer dereference will happen in ReadDirectoryEntry().
    Add additional check to resolve.
    
    Cc: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Cc: Ruiyu Ni <ruiyu.ni@xxxxxxxxx>
    Cc: Star Zeng <star.zeng@xxxxxxxxx>
    Cc: Eric Dong <eric.dong@xxxxxxxxx>
    Contributed-under: TianoCore Contribution Agreement 1.1
    Signed-off-by: Hao Wu <hao.a.wu@xxxxxxxxx>
    Reviewed-by: Paulo Alcantara <pcacjr@xxxxxxxxx>
    Reviewed-by: Star Zeng <star.zeng@xxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel