Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Xidel 0.9.9 (20210529.7880.e0cb7e033b4b) head dump / memory leak (?) #70

Closed
iegubkin opened this issue Jun 4, 2021 · 1 comment
Closed

Comments

@iegubkin
Copy link

iegubkin commented Jun 4, 2021

With fully updated Arch Linux and default kernel (5.12.9) earlier version of xidel works flawlessly:

$ xidel --version
Xidel 0.9.9
(20210309.7795.c4e17e2d216c)

$ xidel --data=ap.json --silent --extract='$json/cards/contents/headline'

US to swiftly boost global vaccine sharing, Biden announces
Trump's grip on GOP sparks fears about democratic process
No 'provoking': Israeli official vows quieter tone with US
Biden's pledge on media freedom may be easier said than done
...

However the latest release of xidel fails with the output:

$ xidel --version
Xidel 0.9.9
(20210529.7880.e0cb7e033b4b)

Heap dump by heaptrc unit of /usr/bin/xidel
2710 memory blocks allocated : 922108/924128
2710 memory blocks freed     : 922108/924128
0 unfreed memory blocks : 0
True heap size : 196608
True free heap : 196608

$ xidel --data=ap.json --silent --extract='$json/cards/contents/headline'
Error:
err:XPST0008: Unknown variable: $json
Heap dump by heaptrc unit of /usr/bin/xidel
2824 memory blocks allocated : 1536575/1538752
2809 memory blocks freed     : 927191/929352
15 unfreed memory blocks : 609384
True heap size : 1409024
True free heap : 796192
Should be : 796744
Call trace for block $00007FCFAA57ADE0 size 49
Call trace for block $00007FCFAA57ACC0 size 80
  $00000000004690EB
Call trace for block $00007FCFAA0AA110 size 608657
Call trace for block $00007FCFAA551300 size 32
Call trace for block $00007FCFAA550D00 size 24
Call trace for block $00007FCFAA550C00 size 24
Call trace for block $00007FCFAA550B00 size 24
Call trace for block $00007FCFAA543100 size 54
  $000000000046CFE0
Call trace for block $00007FCFAA550200 size 24
  $000000000046CEE4
Call trace for block $00007FCFAA55C180 size 120
  $0000000000465110
Call trace for block $00007FCFAA550700 size 24
  $000000000046CD34
Call trace for block $00007FCFAA550600 size 24
Call trace for block $00007FCFAA550500 size 24
Call trace for block $00007FCFAA550300 size 32
Call trace for block $00007FCFAA5E3AC0 size 192

Thank you for this most-useful program. Let me know if I can be of further help.

@benibela
Copy link
Owner

benibela commented Jun 5, 2021

Somehow it became a debug build with memory leak and uninitialized variable detection enabled. And latter breaking the json format detection

benibela added a commit that referenced this issue Jun 20, 2021
the values were correctly overriden by out without -gl. Now override them explicitly rather than relying on out
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants