guysoft / CustomPiOS

A Raspberry Pi and other ARM devices distribution builder
GNU General Public License v3.0
508 stars 146 forks source link

permissions of the generated image file #211

Closed umlaeute closed 11 months ago

umlaeute commented 11 months ago

the generated image file is readable, writable and executable by everybody:

$ ls -lha workspace/*.img
-rwxrwxrwx 1 root     root      14G Oct 17 15:05 workspace/custompios.img

is this intended?

in general i heartily:

the culprit can be found in https://github.com/guysoft/CustomPiOS/blob/55a92830d166ba1f9c6702a3d9d8a20ed416f723/src/custompios#L207

i think this should be changed to

chmod 644 "${BASE_IMG_PATH}"
guysoft commented 11 months ago

I guess we could change that. Sounds reasonable. My flows need it readable here by other users but I can just add a chmod at the end.

umlaeute commented 11 months ago
guysoft commented 11 months ago

I agreed with you :) You are welcome to send a PR