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.

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

openbsd_privdrop.py 0.1.2: add fattr to pledge_promises #560

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

oxzi
Copy link
Contributor

@oxzi oxzi commented Aug 19, 2024

Script info

  • Script name: openbsd_privdrop.py
  • Version: 0.1.1 -> 0.1.2

Description

Since WeeChat 4.3.0, the return value of chmod is being checked1. This revealed a too strict pledge promise, missing fattr.

As an additional change, the level of documentation was increased, including a remainder why read permissions on /home are necessary.

Checklist (script update)

  • Author has been contacted (NOTE: I am the author)
  • Single commit, single file added
  • Commit message format: script_name.py X.Y: …
  • Script version and Changelog have been updated
  • For Python script: works with Python 3 (Python 2 support is optional)
  • Score 100 / 100 displayed by weechat-script-lint

Footnotes

  1. https://github.com/weechat/weechat/commit/2423fdbf2d66d241c809f797f75afd65dff91568#diff-b08b83cf9b86fa0f5c4ca579554a7ba593a892fd143526210844a05701e57058

Since WeeChat 4.3.0, the return value of chmod is being checked[0]. This
revealed a too strict pledge promise, missing fattr.

As an additional change, the level of documentation was increased,
including a remainder why read permissions on /home are necessary.

[0]: weechat/weechat@2423fdb#diff-b08b83cf9b86fa0f5c4ca579554a7ba593a892fd143526210844a05701e57058
@flashcode flashcode self-assigned this Aug 19, 2024
@flashcode flashcode added the bug Unexpected problem or unintended behavior label Aug 19, 2024
@flashcode flashcode merged commit e29e776 into weechat:master Aug 19, 2024
1 check passed
@flashcode
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants