* [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode
@ 2022-04-19 18:22 Luiz Augusto von Dentz
2022-04-19 18:22 ` [PATCH BlueZ 2/2] bluetooth.service: Set ConfigurationDirectoryMode Luiz Augusto von Dentz
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Luiz Augusto von Dentz @ 2022-04-19 18:22 UTC (permalink / raw)
To: linux-bluetooth
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
This sets StateDirectoryMode to 0700 as it is the current mode used for
creating files inside the storage and it is different than the default
systemd uses which is 0755:
[1] https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectoryMode=
---
src/bluetooth.service.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/bluetooth.service.in b/src/bluetooth.service.in
index 4d39ad49d..4ea98b506 100644
--- a/src/bluetooth.service.in
+++ b/src/bluetooth.service.in
@@ -20,6 +20,7 @@ PrivateTmp=true
ProtectKernelTunables=true
ProtectControlGroups=true
StateDirectory=bluetooth
+StateDirectoryMode=0700
ConfigurationDirectory=bluetooth
# Execute Mappings
--
2.35.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH BlueZ 2/2] bluetooth.service: Set ConfigurationDirectoryMode
2022-04-19 18:22 [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode Luiz Augusto von Dentz
@ 2022-04-19 18:22 ` Luiz Augusto von Dentz
2022-04-19 21:03 ` [BlueZ,1/2] bluetooth.service: Set StateDirectoryMode bluez.test.bot
2022-04-20 1:10 ` [PATCH BlueZ 1/2] " patchwork-bot+bluetooth
2 siblings, 0 replies; 4+ messages in thread
From: Luiz Augusto von Dentz @ 2022-04-19 18:22 UTC (permalink / raw)
To: linux-bluetooth
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
This sets ConfigurationDirectoryMode to 0555 to really enforce the
ConfigurationDirectory to be read-only [1].
[1] https://github.com/bluez/bluez/issues/329#issuecomment-1102459104
---
src/bluetooth.service.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/bluetooth.service.in b/src/bluetooth.service.in
index 4ea98b506..beb98ce0c 100644
--- a/src/bluetooth.service.in
+++ b/src/bluetooth.service.in
@@ -22,6 +22,7 @@ ProtectControlGroups=true
StateDirectory=bluetooth
StateDirectoryMode=0700
ConfigurationDirectory=bluetooth
+ConfigurationDirectoryMode=0555
# Execute Mappings
MemoryDenyWriteExecute=true
--
2.35.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [BlueZ,1/2] bluetooth.service: Set StateDirectoryMode
2022-04-19 18:22 [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode Luiz Augusto von Dentz
2022-04-19 18:22 ` [PATCH BlueZ 2/2] bluetooth.service: Set ConfigurationDirectoryMode Luiz Augusto von Dentz
@ 2022-04-19 21:03 ` bluez.test.bot
2022-04-20 1:10 ` [PATCH BlueZ 1/2] " patchwork-bot+bluetooth
2 siblings, 0 replies; 4+ messages in thread
From: bluez.test.bot @ 2022-04-19 21:03 UTC (permalink / raw)
To: linux-bluetooth, luiz.dentz
[-- Attachment #1: Type: text/plain, Size: 2273 bytes --]
This is automated email and please do not reply to this email!
Dear submitter,
Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=633488
---Test result---
Test Summary:
CheckPatch FAIL 1.24 seconds
GitLint FAIL 0.89 seconds
Prep - Setup ELL PASS 50.22 seconds
Build - Prep PASS 0.62 seconds
Build - Configure PASS 9.80 seconds
Build - Make PASS 1740.83 seconds
Make Check PASS 12.41 seconds
Make Check w/Valgrind PASS 530.59 seconds
Make Distcheck PASS 274.43 seconds
Build w/ext ELL - Configure PASS 9.91 seconds
Build w/ext ELL - Make PASS 1705.33 seconds
Incremental Build with patchesPASS 3479.72 seconds
Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[BlueZ,1/2] bluetooth.service: Set StateDirectoryMode
WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#84:
[1] https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectoryMode=
/github/workspace/src/12819297.patch total: 0 errors, 1 warnings, 7 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
/github/workspace/src/12819297.patch has style problems, please review.
NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
##############################
Test: GitLint - FAIL
Desc: Run gitlint with rule in .gitlint
Output:
[BlueZ,1/2] bluetooth.service: Set StateDirectoryMode
9: B1 Line exceeds max length (92>80): "[1] https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectoryMode="
---
Regards,
Linux Bluetooth
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode
2022-04-19 18:22 [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode Luiz Augusto von Dentz
2022-04-19 18:22 ` [PATCH BlueZ 2/2] bluetooth.service: Set ConfigurationDirectoryMode Luiz Augusto von Dentz
2022-04-19 21:03 ` [BlueZ,1/2] bluetooth.service: Set StateDirectoryMode bluez.test.bot
@ 2022-04-20 1:10 ` patchwork-bot+bluetooth
2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+bluetooth @ 2022-04-20 1:10 UTC (permalink / raw)
To: Luiz Augusto von Dentz; +Cc: linux-bluetooth
Hello:
This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:
On Tue, 19 Apr 2022 11:22:36 -0700 you wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>
> This sets StateDirectoryMode to 0700 as it is the current mode used for
> creating files inside the storage and it is different than the default
> systemd uses which is 0755:
>
> [1] https://www.freedesktop.org/software/systemd/man/systemd.exec.html#RuntimeDirectoryMode=
>
> [...]
Here is the summary with links:
- [BlueZ,1/2] bluetooth.service: Set StateDirectoryMode
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=edc69d253445
- [BlueZ,2/2] bluetooth.service: Set ConfigurationDirectoryMode
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=00cfb36e20e3
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-04-20 1:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-19 18:22 [PATCH BlueZ 1/2] bluetooth.service: Set StateDirectoryMode Luiz Augusto von Dentz
2022-04-19 18:22 ` [PATCH BlueZ 2/2] bluetooth.service: Set ConfigurationDirectoryMode Luiz Augusto von Dentz
2022-04-19 21:03 ` [BlueZ,1/2] bluetooth.service: Set StateDirectoryMode bluez.test.bot
2022-04-20 1:10 ` [PATCH BlueZ 1/2] " patchwork-bot+bluetooth
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.