1. 07 7月, 2021 15 次提交
  2. 06 7月, 2021 11 次提交
    • T
      Merge branch '2021-07-01-update-CI-containers' · 1311dd37
      Tom Rini 提交于
      - General test.py improvements
      - Rewrite the squashfs tests
      - Update our CI container to Ubuntu 20.04 "focal" base.
      - Make some changes to the Azure yaml so that we can have more tests run
        there.
      1311dd37
    • T
      Docker/CI: Update to "focal" and latest build · b1c2102d
      Tom Rini 提交于
      Move us up to being based on Ubuntu 20.04 "focal" and the latest tag
      from Ubuntu for this release.  For this, we make sure that "python" is
      now python3 but still include python2.7 for the rx51 qemu build as that
      is very old and does not support python3.
      Signed-off-by: NTom Rini <trini@konsulko.com>
      b1c2102d
    • J
      test/py: rewrite sqfsls command test suite · 9bde9b5e
      Joao Marcos Costa 提交于
      Add more details to test cases by comparing each expected line with the
      command's output. Add new test cases:
      - sqfsls at an empty directory
      - sqfsls at a sub-directory
      Reviewed-by: NSimon Glass <sjg@chromium.org>
      Tested-by: Simon Glass <sjg@chromium.org> [on sandbox]
      Signed-off-by: NJoao Marcos Costa <jmcosta944@gmail.com>
      9bde9b5e
    • J
      test/py: rewrite sqfsload command test suite · 208eb2a4
      Joao Marcos Costa 提交于
      The previous strategy to know if a file was correctly loaded was to
      check for how many bytes were read and compare it against the file's
      original size. Since this is not a good solution, replace it by
      comparing the checksum of the loaded bytes against the original file's
      checksum. Add more test cases: files at a sub-directory and non-existent
      file.
      Reviewed-by: NSimon Glass <sjg@chromium.org>
      Tested-by: Simon Glass <sjg@chromium.org> [on sandbox]
      Signed-off-by: NJoao Marcos Costa <jmcosta944@gmail.com>
      208eb2a4
    • J
      test/py: rewrite common tools for SquashFS tests · 04c9813e
      Joao Marcos Costa 提交于
      Remove the previous OOP approach, which was confusing and incomplete.
      Add more test cases by making SquashFS images with various options,
      concerning file fragmentation and its compression. Add comments to
      properly document the code.
      Reviewed-by: NSimon Glass <sjg@chromium.org>
      Tested-by: Simon Glass <sjg@chromium.org> [on sandbox]
      Signed-off-by: NJoao Marcos Costa <jmcosta944@gmail.com>
      04c9813e
    • A
      Azure: Add loop devices and CAP_SYS_ADMIN for sandbox test.py tests · e22ec9c6
      Alper Nebi Yasak 提交于
      The filesystem test setup needs to prepare disk images for its tests,
      with either guestmount or loop mounts. The former requires access to the
      host fuse device (added in a previous patch), the latter requires access
      to host loop devices. Both mounts also need additional privileges since
      docker's default configuration prevents the containers from mounting
      filesystems (for host security).
      
      Add any available loop devices to the container and try to add as few
      privileges as possible to run these tests, which narrow down to adding
      SYS_ADMIN capability and disabling apparmor confinement. However, this
      much still seems to be insecure enough to let malicious container
      processes escape as root on the host system [1].
      
      [1] https://blog.trailofbits.com/2019/07/19/understanding-docker-container-escapes/
      
      Since the mentioned tests are marked to run only on the sandbox board,
      add these additional devices and privileges only when testing with that.
      
      An alternative to using mounts is modifying the filesystem tests to use
      virt-make-fs (like some EFI tests do), but it fails to generate a
      partitionless FAT filesystem image on Debian systems. Other more
      feasible alternatives are using guestfish or directly using libguestfs
      Python bindings to create and populate the images, but switching the
      test setups to these is nontrivial and is left as future work.
      Signed-off-by: NAlper Nebi Yasak <alpernebiyasak@gmail.com>
      e22ec9c6
    • A
      Azure: Add fuse device for test.py tests · 1aaaf60d
      Alper Nebi Yasak 提交于
      The EFI secure boot and capsule test setups need to prepare disk images
      for their tests using virt-make-fs, which requires access to the host
      fuse device. This is not exposed to the docker container by default and
      has to be added explicitly. Add it.
      Signed-off-by: NAlper Nebi Yasak <alpernebiyasak@gmail.com>
      1aaaf60d
    • A
      tools: docker: Install a readable kernel for libguestfs-tools · f9abaa53
      Alper Nebi Yasak 提交于
      The filesystem and EFI (capsule and secure boot) test setups try to use
      guestmount and virt-make-fs respectively to prepare disk images to run
      tests on. However, these libguestfs tools need a kernel image and fail
      with the following message (revealed in debug/trace mode) if it can't
      find one:
      
          supermin: failed to find a suitable kernel (host_cpu=x86_64).
      
          I looked for kernels in /boot and modules in /lib/modules.
      
          If this is a Xen guest, and you only have Xen domU kernels
          installed, try installing a fullvirt kernel (only for
          supermin use, you shouldn't boot the Xen guest with it).
      
      This failure then causes these tests to be skipped in CIs. Install a
      kernel package in the Docker containers so the CIs can run these
      tests with libguestfs tools again (assuming the container is run with
      necessary host devices and privileges). As this kernel would be only
      used for virtualization, we can use the kernel package specialized for
      that. On Ubuntu systems kernel images are not readable by non-root
      users, so explicitly add read permissions with chmod as well.
      Signed-off-by: NAlper Nebi Yasak <alpernebiyasak@gmail.com>
      Acked-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      f9abaa53
    • A
      test/py: Wait for guestmount worker to exit after running guestunmount · 99f5303c
      Alper Nebi Yasak 提交于
      Some filesystem tests are failing when their image is prepared with
      guestmount, but succeeding if loop mounts are used instead. The reason
      seems to be a race condition the guestmount(1) manual page explains:
      
          When guestunmount(1)/fusermount(1) exits, guestmount may still be
          running and cleaning up the mountpoint.  The disk image will not be
          fully finalized.
      
          This means that scripts like the following have a nasty race condition:
      
           guestmount -a disk.img -i /mnt
           # copy things into /mnt
           guestunmount /mnt
           # immediately try to use 'disk.img' ** UNSAFE **
      
          The solution is to use the --pid-file option to write the guestmount
          PID to a file, then after guestunmount spin waiting for this PID to
          exit.
      
      The Python standard library has an os.waitpid() function for waiting a
      child to terminate, but it cannot wait on non-child processes. Implement
      a utility function that can do this by polling the process repeatedly
      for a given duration, optionally killing the process if it won't
      terminate on its own. Apply the suggested solution with this utility
      function, which makes the failing tests succeed again.
      Signed-off-by: NAlper Nebi Yasak <alpernebiyasak@gmail.com>
      Reviewed-by: NSimon Glass <sjg@chromium.org>
      99f5303c
    • A
      test/py: Use loop mounts if guestmount fails in filesystem tests · 8f5f5d3a
      Alper Nebi Yasak 提交于
      If guestmount isn't available on the system, filesystem test setup falls
      back to using loop mounts to prepare its disk images. If guestmount is
      available but fails to work, the tests are immediately skipped. Instead
      of giving up on a guestmount failure, try using loop mounts as an
      attempt to keep tests running.
      
      Also stop checking if guestmount is in PATH, as trying to run a missing
      guestmount can now follow the same failure codepath and fall back to
      loop mounts anyway.
      Signed-off-by: NAlper Nebi Yasak <alpernebiyasak@gmail.com>
      8f5f5d3a
    • A
      env: efi: fix a wrong address dereference · f1eb346e
      AKASHI Takahiro 提交于
      Probably, a pointer to a variable in an inner block should not
      be exposed to an outer block.
      
      Fixes: c70f4481 ("efi_loader: simplify 'printenv -e'")
      Signed-off-by: NAKASHI Takahiro <takahiro.akashi@linaro.org>
      [trini: Don't make guid const now]
      Signed-off-by: NTom Rini <trini@konsulko.com>
      f1eb346e
  3. 05 7月, 2021 2 次提交
  4. 03 7月, 2021 5 次提交
  5. 02 7月, 2021 7 次提交
    • H
      efi_loader: clear OsIndications · 149108a3
      Heinrich Schuchardt 提交于
      After each reboot we must clear flag
      EFI_OS_INDICATIONS_FILE_CAPSULE_DELIVERY_SUPPORTED in variable
      OsIndications.
      Signed-off-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      149108a3
    • M
      efi_loader: Improve the parameter check for QueryVariableInfo() · 417a3c24
      Masami Hiramatsu 提交于
      Improve efi_query_variable_info() to check the parameter settings and
      return correct error code according to the UEFI Specification 2.9,
      and the Self Certification Test (SCT) II Case Specification, June
      2017, chapter 4.1.4 QueryVariableInfo().
      Reported-by: NKazuhiko Sakamoto <sakamoto.kazuhiko@socionext.com>
      Signed-off-by: NMasami Hiramatsu <masami.hiramatsu@linaro.org>
      Reviewed-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      417a3c24
    • H
      efi_loader: missing parentheses in query_console_size · abd62e4d
      Heinrich Schuchardt 提交于
      After if we should use parentheses to keep the code readable.
      
      Fixes: a95f4c88 ("efi_loader: NULL dereference in EFI console")
      Signed-off-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      abd62e4d
    • I
      efi_loader: Always install FMPs · 6e0184b8
      Ilias Apalodimas 提交于
      We only install FMPs if a CapsuleUpdate is requested.  Since we now have an
      ESRT table which relies on FMPs to build the required information, it
      makes more sense to unconditionally install them. This will allow userspace
      applications (e.g fwupd) to make use of the ERST and provide us with files
      we can use to run CapsuleUpdate on-disk
      Signed-off-by: NIlias Apalodimas <ilias.apalodimas@linaro.org>
      Reviewed-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      6e0184b8
    • I
      efi_loader: Force a single FMP instance per hardware store · b891ff18
      Ilias Apalodimas 提交于
      Chapter 23 of the EFI spec (rev 2.9) says:
      "A specific updatable hardware firmware store must be represented by
      exactly one FMP instance".
      This is not the case for us, since both of our FMP protocols can be
      installed at the same time because they are controlled by a single
      'dfu_alt_info' env variable.
      So make the config options depend on each other and allow the user to
      install one of them at any given time.  If we fix the meta-data provided
      by the 'dfu_alt_info' in the future,  to hint about the capsule type
      (fit or raw) we can revise this and enable both FMPs to be installed, as
      long as they target different firmware hardware stores
      
      Note that we are not using a Kconfig 'choice' on purpose, since we
      want to allow both of those to be installed and tested in sandbox
      Signed-off-by: NIlias Apalodimas <ilias.apalodimas@linaro.org>
      Reviewed-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      b891ff18
    • M
      efi: Fix to use null handle to create new handle for efi_fmp_raw · 3ef77223
      Masami Hiramatsu 提交于
      When running the efidebug capsule disk-update command, the efi_fmp_raw
      protocol installation fails with 2 (EFI_INVALID_PARAMETER) as below.
      This is because the code passes efi_root instead of the handle local var.
      
      => efidebug capsule disk-update
      EFI: Call: efi_install_multiple_protocol_interfaces( &handle, &efi_guid_firmware_management_protocol, &efi_fmp_fit, NULL)
        EFI: Entry efi_install_multiple_protocol_interfaces(00000000fbaf5988)
          EFI: Call: efi_install_protocol_interface( handle, protocol, EFI_NATIVE_INTERFACE, protocol_interface)
            EFI: Entry efi_install_protocol_interface(00000000fbaf5988, 86c77a67-0b97-4633-a187-49104d0685c7, 0, 00000000fbfa6ee8)
              EFI: new handle 00000000fbb37520
            EFI: Exit: efi_install_protocol_interface: 0
          EFI: 0 returned by efi_install_protocol_interface( handle, protocol, EFI_NATIVE_INTERFACE, protocol_interface)
        EFI: Exit: efi_install_multiple_protocol_interfaces: 0
      EFI: 0 returned by efi_install_multiple_protocol_interfaces( &handle, &efi_guid_firmware_management_protocol, &efi_fmp_fit, NULL)
      EFI: Call: efi_install_multiple_protocol_interfaces( &efi_root, &efi_guid_firmware_management_protocol, &efi_fmp_raw, NULL)
        EFI: Entry efi_install_multiple_protocol_interfaces(00000000fbfec648)
          EFI: Call: efi_install_protocol_interface( handle, protocol, EFI_NATIVE_INTERFACE, protocol_interface)
            EFI: Entry efi_install_protocol_interface(00000000fbfec648, 86c77a67-0b97-4633-a187-49104d0685c7, 0, 00000000fbfa6f18)
              EFI: handle 00000000fbaf8520
            EFI: Exit: efi_install_protocol_interface: 2
          EFI: 2 returned by efi_install_protocol_interface( handle, protocol, EFI_NATIVE_INTERFACE, protocol_interface)
        EFI: Exit: efi_install_multiple_protocol_interfaces: 2
      EFI: 2 returned by efi_install_multiple_protocol_interfaces( &efi_root, &efi_guid_firmware_management_protocol, &efi_fmp_raw, NULL)
      Command failed, result=1
      
      To fix this issue, pass the handle local var which is set NULL right
      before installing efi_fmp_raw as same as the installing efi_fmp_fit.
      (In both cases, the local reference to the handle will be just discarded)
      Signed-off-by: NMasami Hiramatsu <masami.hiramatsu@linaro.org>
      Signed-off-by: NIlias Apalodimas <ilias.apalodimas@linaro.org>
      Reviewed-by: NHeinrich Schuchardt <xypron.glpk@gmx.de>
      3ef77223
    • T
      Merge branch '2021-07-01-buildtime-gd-sanity-check' into next · 03b61ffe
      Tom Rini 提交于
      - Merge build-time sanity checks to ensure the size of gd doesn't
        change.  This can happen during cleanups due to not all symbols being
        implemented in Kconfig.
      03b61ffe