search

tpm2-software / tpm2-totp

Attest the trustworthiness of a device against a human using time-based one-time passwords
https://tpm2-software.github.io
BSD 3-Clause "New" or "Revised" License
163 stars 36 forks source link

src: fix compilation failure due to "variable may be used uninitialized" #33

Closed diabonas closed 5 years ago

diabonas commented 5 years ago

Apply the patch provided by @casantos to fix #32.

diabonas commented 5 years ago

Should we cherry-pick this for v0.1.x as well? I notice there is still a 0.1.2 release candidate from 2019-04-08. If we do another patch release before 1.0.0, #31 should be backported as well.

jacmet commented 5 years ago

@diabonas I would appreciate it getting applied to v0.1.x as well

AndreasFuchsTPM commented 5 years ago

We have an RC ? I'll update it later with this patch...

codecov[bot] commented 5 years ago

Codecov Report

Merging #33 into master will increase coverage by 0.02%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   80.46%   80.48%   +0.02%     
==========================================
  Files           5        5              
  Lines         819      820       +1     
==========================================
+ Hits          659      660       +1     
  Misses        160      160
Impacted Files Coverage Δ
src/tpm2-totp.c 82.12% <100%> (+0.08%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a3dba24...8d99000. Read the comment docs.