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

Fix errors in win7 sp1 x64 #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

breezechen
Copy link

No description provided.

@redxu
Copy link
Owner

redxu commented Aug 5, 2016

具体是什么错误?编译上的错误还是运行上的错误?
为什么要用detours替代hookapi

@breezechen
Copy link
Author

哦 是这样的, 既然你的项目里面用了detours, 那么挂钩子的事情就全部让detours来做吧, 所以我改了一下.

@breezechen
Copy link
Author

我的系统是win7 64 sp1, 错误原因是loaderdll.dll并没有导出函数, 导致Insight3.exe运行失败, 所以我加了一个空的按序号导出的函数.

@redxu
Copy link
Owner

redxu commented Aug 5, 2016

我的系统也是win7 x64 sp1,没遇到你说的因为loaderdll没导出函数导致运行失败的情况,有错误截图吗?

@breezechen
Copy link
Author


我是用vs2013编译的

@breezechen
Copy link
Author

@redxu
Copy link
Owner

redxu commented Aug 5, 2016

错误看上去像内存或者运行库的问题,印象中如果是因为没有导出函数会报一个没有导出函数的错误.
另外本项目编译发布的工具是mingw-gcc,vs的c编译器对c标准的支持很不好.标准c中默认没加限制的函数都是自动导出的.
不用detours而用hookapi是因为会增大发布文件的大小,hookapi也足够精致小巧耐用.
所以,你可以单独PR一个VS2013的工程文件.

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

Successfully merging this pull request may close these issues.

2 participants