Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 8beddaf) | patch
vppinfra: change fchmod to umask for unix socket 62/39862/3
authorGeorgy Borodin <bor1-go@yandex-team.ru>
Fri, 10 Nov 2023 15:31:09 +0000 (16:31 +0100)
committerDamjan Marion <dmarion@0xa5.net>
Wed, 17 Jan 2024 19:10:13 +0000 (19:10 +0000)
commitdc26d50426792954e372cb7949b94fd3eb573942
treee814fe3de6e2ed7ad8e88909d2d688b708538ee8tree | snapshot
parent8beddaf5b435a872b844052b506366b7a474004ccommit | diff
vppinfra: change fchmod to umask for unix socket

Setting g+w permission for unix sockets didn't work. There were
two problems:
1. new flag local_only wasn't set for all AF_UNIX sockets;
2. fchmod is not a good choice for sockets.

fchmod was replaced with couple of umasks, and local_only with
socket type check.

Type: fix
Fixes: 085757bb4930511928daa97f972cdca021e7a813
Change-Id: I8dc0fceb110a36bfa234f552bbdf182e09e55e27
Signed-off-by: Georgy Borodin <bor1-go@yandex-team.ru>
src/vppinfra/socket.c diff | blob | history
Vector Packet Processing