EDAC: Fix wrong executable file modes for C source files

[PATCH] EDAC: Fix wrong executable file modes for C source files

Posted by Kuan-Wei Chiu 1 month ago

Three EDAC source files were mistakenly marked as executable in
commit 1e14ea901dc8 ("EDAC: Initialize EDAC features sysfs attributes").

These are plain C source files and should not carry the executable bit.
Correcting their modes follows the principle of least privilege and
avoids unnecessary execute permissions in the repository.

Fixes: 1e14ea901dc8 ("EDAC: Initialize EDAC features sysfs attributes")
Signed-off-by: Kuan-Wei Chiu <visitorckw@gmail.com>
---
 drivers/edac/ecs.c        | 0
 drivers/edac/mem_repair.c | 0
 drivers/edac/scrub.c      | 0
 3 files changed, 0 insertions(+), 0 deletions(-)
 mode change 100755 => 100644 drivers/edac/ecs.c
 mode change 100755 => 100644 drivers/edac/mem_repair.c
 mode change 100755 => 100644 drivers/edac/scrub.c

diff --git a/drivers/edac/ecs.c b/drivers/edac/ecs.c
old mode 100755
new mode 100644
diff --git a/drivers/edac/mem_repair.c b/drivers/edac/mem_repair.c
old mode 100755
new mode 100644
diff --git a/drivers/edac/scrub.c b/drivers/edac/scrub.c
old mode 100755
new mode 100644
-- 
2.34.1

Re: [PATCH] EDAC: Fix wrong executable file modes for C source files

Posted by Borislav Petkov 1 month ago

On Fri, Aug 29, 2025 at 03:19:54AM +0800, Kuan-Wei Chiu wrote:
> Three EDAC source files were mistakenly marked as executable in
> commit 1e14ea901dc8 ("EDAC: Initialize EDAC features sysfs attributes").
> 
> These are plain C source files and should not carry the executable bit.
> Correcting their modes follows the principle of least privilege and
> avoids unnecessary execute permissions in the repository.
> 
> Fixes: 1e14ea901dc8 ("EDAC: Initialize EDAC features sysfs attributes")
> Signed-off-by: Kuan-Wei Chiu <visitorckw@gmail.com>
> ---
>  drivers/edac/ecs.c        | 0
>  drivers/edac/mem_repair.c | 0
>  drivers/edac/scrub.c      | 0
>  3 files changed, 0 insertions(+), 0 deletions(-)
>  mode change 100755 => 100644 drivers/edac/ecs.c
>  mode change 100755 => 100644 drivers/edac/mem_repair.c
>  mode change 100755 => 100644 drivers/edac/scrub.c
> 
> diff --git a/drivers/edac/ecs.c b/drivers/edac/ecs.c
> old mode 100755
> new mode 100644
> diff --git a/drivers/edac/mem_repair.c b/drivers/edac/mem_repair.c
> old mode 100755
> new mode 100644
> diff --git a/drivers/edac/scrub.c b/drivers/edac/scrub.c
> old mode 100755
> new mode 100644
> -- 

Applied, thanks.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette