OFS / opae-sdk

Open Programmable Acceleration Engine
https://ofs.github.io
BSD 3-Clause "New" or "Revised" License
258 stars 82 forks source link

Update: use json to save opae.io driver state #3028

Closed tswhison closed 1 year ago

tswhison commented 1 year ago

Description

opae.io init unbinds the target device from its driver and writes a breadcrumb out to file so that the driver can be restored during opae.io release. Previously, we were using the pickle module to do this.

This github issue #3024 was filed, noting that pickle is prone to code execution injection issues. The proposal was to use json instead.

Collateral (docs, reports, design examples, case IDs):

3024

Tests added:

Tests run:

Manual verification using opae.io init followed by opae.io release.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 6502084603


Files with Coverage Reduction New Missed Lines %
binaries/fpgad/monitor_thread.c 1 85.57%
libraries/plugins/xfpga/mmio.c 2 84.72%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 6501241052: 0.05%
Covered Lines: 15768
Relevant Lines: 23053

💛 - Coveralls