-
Notifications
You must be signed in to change notification settings - Fork 8
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
Sigkit examples are broken. #24
Comments
Hi Dustin, The examples in the repository are indeed old. I believe I created them three years ago as part of a summer intern project, and the scripts were useful to illustrate how (at the time) they could be used to generate signatures for Ubuntu glibc functions. You are welcome to contribute fixes for them, although the code is no longer under my maintenance. Best, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The libc-scraper for sigkit is broken.
The awk in run.sh "needs" to be $4, not $3.
parallel's -j argument should probably be 0, not 3.
run.sh should probably use an absolute or relative path to process-deb.sh, instead of just assuming it will be in $PATH (unless an install script is shipped).
It should probably be documented that process-deb.sh depends on batch_process.py being in ~/sigkit/ (unless an install script is shipped).
pkl_blacklist in merge_ubuntu.py should probably be a list literal instead of a dictionary literal.
merge_ubuntu should probably operate on more than just bionic in main.
The documentation suggests merge_ubuntu.py uses all the cpus available on the machine. The cpu_factor, which is passed to the ProcessPoolExecutor constructor only uses the square root of the number of CPUs.
convert_siglib.py should probably use os.path.basename and not indexing on ".", to catch relative paths.
I'm happy to do a PR for some of these, if some decisions are made about the right way to handle them.
The text was updated successfully, but these errors were encountered: