From: Thomas Hebb Date: Sun, 1 Mar 2020 18:47:53 +0000 (-0800) Subject: mkimage: fit: don't create image with 0700 permissions X-Git-Tag: v2020.07-rc1~27^2~16 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=ab5a2b0f7927196143d762b2ad9cdc51b84249b3;p=oweals%2Fu-boot.git mkimage: fit: don't create image with 0700 permissions commit 7298e422504e ("mkimage: fit: add support to encrypt image with aes") added a new copyfile() function as part of the FIT image creation flow. This function as currently written creates the final image with a mode of 0700 (before umask), differing from the old behavior of 0666. Since there doesn't seem to be any reason to make the image executable or non-group, non-other readable, change the mask to 0666 to preserve the old behavior. Fixes: 7298e422504e ("mkimage: fit: add support to encrypt image with aes") Signed-off-by: Thomas Hebb --- diff --git a/tools/fit_image.c b/tools/fit_image.c index dd61a816c9..4301b5decb 100644 --- a/tools/fit_image.c +++ b/tools/fit_image.c @@ -642,7 +642,7 @@ static int copyfile(const char *src, const char *dst) goto out; } - fd_dst = open(dst, O_WRONLY | O_CREAT, 0700); + fd_dst = open(dst, O_WRONLY | O_CREAT, 0666); if (fd_dst < 0) { printf("Can't open file %s (%s)\n", dst, strerror(errno)); goto out;