search

Snaipe / Mimick

A KISS, cross-platform C mocking library
MIT License
152 stars 43 forks source link

Can't find vital functions 'abort' 'vfprintf' #18

Open Cynerd opened 3 years ago

Cynerd commented 3 years ago

I am trying to use Mimick on Gentoo. I am not sure if this is generic issue of just Gentoo specific one as I have not tried it on any other distribution. I tried both latest master as well as v0.3.0 tag, it behaves the same way.

Simply even running mmk_test results in message:

mimick: Initialization error: could not find definitions for vital function(s): 'abort' 'vfprintf'

I encounter same behavior of course not only with test but every time I try to use mimick.

mwhipple commented 3 years ago

I'm also running into this error and happen to also be running Gentoo. I'll be looking to pick through it and collect more info over the next several days.

mwhipple commented 3 years ago

My current theory is that the high level of customization for build options in Gentoo is running afoul of some of the expectations of this library, where some of the glibc libraries may not be using the symbol hash format expected here. I'll be continuing to research and tinker though will somewhat be waiting and watching compilations while I work on other things.

mwhipple commented 3 years ago

The above theory seems to be the case on my system at least: Mimick is written to look for the symbol table in a library with the DT_HASH tag which is presumably the sysv style hash format whereas my libc.so only has a DT_GNU_HASH which apparently includes some optimization around missing elements. Mimick should likely be adjusted to use either format though the --hash-style=both build argument could likely also help. I'll probably try to work on getting a patch together.

peadar commented 3 years ago

Pull request #26 implements gnu_hash lookups - Can you see if the "gnuhash" branch on peadar/mimick fixes this?

mwhipple commented 3 years ago

awesome, thanks! I'll try it out in the next day.

mwhipple commented 3 years ago

The branch provided works well for me. @Cynerd?

Cynerd commented 3 years ago

The test passes so I would say that it is fixed for me as well. Thank you very much. I am enthusiastically awaiting merge and new release. Thank you for fixing this.