noseglasses / elf_diff

A tool to compare ELF binaries
GNU General Public License v3.0
171 stars 21 forks source link

Fixed typo when setting Symbol type_ #83

Closed warnesp closed 2 years ago

warnesp commented 2 years ago

Was causing the symbol type to show as '?' in reports because the default value was not being set.

noseglasses commented 2 years ago

Great, thanks!

codecov-commenter commented 2 years ago

Codecov Report

Merging #83 (b04d084) into master (2ba7e49) will increase coverage by 0.08%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   88.63%   88.71%   +0.08%     
==========================================
  Files          29       29              
  Lines        2331     2331              
  Branches      320      320              
==========================================
+ Hits         2066     2068       +2     
+ Misses        169      167       -2     
  Partials       96       96              
Impacted Files Coverage Δ
src/elf_diff/binary.py 91.66% <100.00%> (ø)
src/elf_diff/git.py 64.70% <0.00%> (+11.76%) :arrow_up: