search

C-RH-C / crhc-cli

C.RH.C API Command Line Tool
GNU General Public License v3.0
18 stars 13 forks source link

crhc-linux-x64 binary won't work if /tmp is mounted with noexec option #183

Open sayan3296 opened 1 year ago

sayan3296 commented 1 year ago

The crhc script or the crhc-linux-x64 binary itself will not work and throw the following error if /tmp is mounted with noexec option in that system:

# ./crhc-linux-x64
./crhc-linux-x64: error while loading shared libraries: libz.so.1: failed to map segment from shared object

https://github.com/C-RH-C/crhc-cli/blob/master/conf/conf.py#L8-L17 forces\defaults the directory to be /tmp.

Hence. Either the error handling needs to be improved here or an option should be introduced to pass a different tmpdir for crhc .

sayan3296 commented 1 year ago

CC @waldirio

waldirio commented 1 year ago

Hello @sayan3296

Thank you for the heads up and sorry for my delay here.

I believe we can add a flag that would change this directory, if necessary, and by default, it will be using the standard. What do you think?

Meanwhile, I'll add a note on the kcs/article here.

Thank you again! Waldirio

sayan3296 commented 1 year ago

Yeah.. A flag sounds perfect.. As long as user can specify a directory when default one cannot be used, we should be good.

waldirio commented 1 year ago

@sayan3296

Perfect. Article update and I'll be working on this flag soon.

If you would like to move on here and push the PR, it will be very welcome!

teamwork++ and thank you in advance! Waldirio