Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

--status file in 2.6.7 #454

Closed
cjw77 opened this issue Nov 14, 2023 · 1 comment
Closed

--status file in 2.6.7 #454

cjw77 opened this issue Nov 14, 2023 · 1 comment
Assignees
@lstipakov

Comments

@cjw77
Copy link

cjw77 commented Nov 14, 2023

In version 2.6.7 the server will not connect for the second time of asking. Options error: --status fails with 'openvpn-status.log': Permission denied (errno=13)

In my config file I have: status openvpn-status.log

openvpn-status.log is created fine the first connection, but listed as read-only.

I uninstalled 2.6.7, and reinstalled the GUI from 2.6.6 together with the service from 2.6.7, same error. I installed only 2.6.6 and now the server can disconnect and reconnect.

OS Windows 11 Pro
@lstipakov lstipakov self-assigned this Nov 14, 2023
@lstipakov
Copy link
Member

Looking into it.

lstipakov added a commit to lstipakov/openvpn that referenced this issue Nov 14, 2023
@lstipakov
config.h: fix incorrect defines for _wopen()
fafb21c 
This is a regression from commit

  0134184 ("add basic CMake based build")

S_IRUSR and S_IWUSR should NOT be defined as 0 but
as _S_IREAD and _S_IWRITE, as it was already fixed in commit

  077445d ("Fix some more wrong defines in config-msvc.h")

Those are used as permission mode when opening a file. Passing
zero makes file read-only, which break for example --status-file
functionality.

Fixes OpenVPN#454

Change-Id: I53eaee85d7b284af6bc63da5f6d8f310ddd96c47
Signed-off-by: Lev Stipakov <lev@openvpn.net>
cron2 pushed a commit to cron2/openvpn that referenced this issue Nov 14, 2023
@lstipakov @cron2
config.h: fix incorrect defines for _wopen()
0c174e4 
This is a regression from commit

  0134184 ("add basic CMake based build")

S_IRUSR and S_IWUSR should NOT be defined as 0 but
as _S_IREAD and _S_IWRITE, as it was already fixed in commit

  077445d ("Fix some more wrong defines in config-msvc.h")

Those are used as permission mode when opening a file. Passing
zero makes file read-only, which break for example --status-file
functionality.

Github: fixes OpenVPN/openvpn#454
Trac: #1430

Change-Id: I53eaee85d7b284af6bc63da5f6d8f310ddd96c47
Signed-off-by: Lev Stipakov <lev@openvpn.net>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20231114141653.10486-1-gert@greenie.muc.de>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg27393.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
(cherry picked from commit 2fcfb77)
@cron2 cron2 closed this as completed in 2fcfb77 Nov 14, 2023
lstipakov added a commit to lstipakov/openvpn-windows-test that referenced this issue Nov 16, 2023
@lstipakov
Add --cd and --status options to config
aa2579c 
This is to make sure we won't get regressions
like

 OpenVPN/openvpn#448
 OpenVPN/openvpn#454

Signed-off-by: Lev Stipakov <lev@openvpn.net>
@lstipakov lstipakov mentioned this issue Nov 16, 2023
Merged
cron2 pushed a commit to OpenVPN/openvpn-windows-test that referenced this issue Nov 17, 2023
@lstipakov @cron2
Add --cd and --status options to config
11cf267 
This is to make sure we won't get regressions
like

 OpenVPN/openvpn#448
 OpenVPN/openvpn#454

Signed-off-by: Lev Stipakov <lev@openvpn.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lstipakov lstipakov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lstipakov @cjw77