m0mchil / poclbm

PyOpenCL bitcoin miner
Other
682 stars 385 forks source link

improper tempfile creation leading to crash #14

Open Skillabstinenz opened 13 years ago

Skillabstinenz commented 13 years ago

poclbm tries to create some sorf of .elf tempfiles in its directory.but this leads to security problems under win7(i have read unter winxp and linux too), which do not allow such behaviour. resulting in the following error:

IOError: [Errno 13] Permission denied: '75f47f8cf8b80b32b9f6eea7786559d4.elf'

to my knowledge such files should be created in the %temp% folder. so this should be fixed quite easily just change the path where the .elf-files are created and recompile the program. i would do it myself but being completely new to github and python makes that quite hard.

rawler commented 13 years ago

I'm not sure why Win7 would disallow that, provided that poclbm isn't installed in some readonly system directory. In any case, it seems like a good idea, so I'm proposing https://github.com/m0mchil/poclbm/pull/39. (Please jamesjames, try it out when you can).