[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 Tobias C. Berner changed: What|Removed |Added Status|New |Closed Resolution|--- |FIXED --- Comment #6 from Tobias C. Berner --- Committed. Thanks you two. -- You are receiving this mail because: You are the assignee for the bug.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 --- Comment #5 from commit-h...@freebsd.org --- A commit references this bug: Author: tcberner Date: Wed Aug 28 18:09:17 UTC 2019 New revision: 510087 URL: https://svnweb.freebsd.org/changeset/ports/510087 Log: security/plasma5-kwallet-pam: make it work - Replace the use of socat with netcat which is in base. Note that netcat is bidirectional. It sends input from stdin over the socket and input from the socket to stdout. After receiving EOF on stdin it keeps polling the socket for input until the other end closes the connection, which in the case of kwalletd never happens because it's also waiting for more data until the other end closes the connection. Call netcat using "nc | :" so its stdout is a pipe of which the far end closes immediately (stdout itself stays open). Without a valid stdout netcat becomes unidirectional. Using "nc >&-" doesn't work because it closes stdout itself and its descriptor will be reused for the socket so netcat thinks it has a valid stdout. - Fix an off-by-one buffer size check. - Fix a call to bind(2). The third argument should be the size of the sockaddr struct. It contains an extra field besides sun_path and sun_family in BSD so the name of the socket got truncated. PR: 228291 Submitted by: tijl, mikael.uran...@gmail.com Changes: head/security/plasma5-kwallet-pam/Makefile head/security/plasma5-kwallet-pam/files/ head/security/plasma5-kwallet-pam/files/patch-pam__kwallet.c head/security/plasma5-kwallet-pam/files/patch-pam__kwallet__init head/security/plasma5-kwallet-pam/pkg-plist -- You are receiving this mail because: You are the assignee for the bug.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 --- Comment #4 from mikael.uran...@gmail.com --- Created attachment 206965 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=206965&action=edit patch regen -- You are receiving this mail because: You are the assignee for the bug.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 Tobias C. Berner changed: What|Removed |Added Flags|maintainer-feedback?(kde@Fr |maintainer-feedback+ |eeBSD.org) | --- Comment #3 from Tobias C. Berner --- Moin moin Sorry, that fell through... could you rebase your patch against the current state? mfg Tobias -- You are receiving this mail because: You are the assignee for the bug.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 --- Comment #2 from Tijl Coosemans --- (In reply to Tobias C. Berner from comment #1) The .kde4 change is behind an ifdef and it should probably be behind an additional ifdef __FreeBSD__. It's only there in case someone wants to make a copy of this port that builds pam_kwallet.so for the kde4 kwalletd. I believe .kde was used by KDE3 on FreeBSD. Dealing with upstream is your job as a maintainer. I don't want to register yet more bugzilla/phabricater/whatever accounts. One thing I forgot to mention: the patch moves the pam module from lib/security to lib where the other pam modules are (see pam.conf(5) about "module-path"). -- You are receiving this mail because: You are the assignee for the bug.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 Tobias C. Berner changed: What|Removed |Added CC||tcber...@freebsd.org --- Comment #1 from Tobias C. Berner --- The change to set kde home to '.kde4' looks bogus. It would be great if you would upstream this, by creating a review on phabricator.kde.org. Mfg Tobias -- You are receiving this mail because: You are the assignee for the bug.
maintainer-feedback requested: [Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
Bugzilla Automation has asked k...@freebsd.org for maintainer-feedback: Bug 228291: [patch] security/plasma5-kwallet-pam: make it work https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 --- Description --- - Remove USES=gettext and add USE_KDE=wallet_run. - Patch the code so kwalletd can be found in LOCALBASE. - Replace the use of socat with netcat which is in base. Note that netcat is bidirectional. It sends input from stdin over the socket and input from the socket to stdout. After receiving EOF on stdin it keeps polling the socket for input until the other end closes the connection, which in the case of kwalletd never happens because it's also waiting for more data until the other end closes the connection. Call netcat using "nc | :" so its stdout is a pipe of which the far end closes immediately (stdout itself stays open). Without a valid stdout netcat becomes unidirectional. Using "nc >&-" doesn't work because it closes stdout itself and its descriptor will be reused for the socket so netcat thinks it has a valid stdout. - Add an upstream followup to the CVE patches committed in ports r469032. - Fix an off-by-one buffer size check. - Fix a call to bind(2). The third argument should be the size of the sockaddr struct. It contains an extra field besides sun_path and sun_family in BSD so the name of the socket got truncated.
[Bug 228291] [patch] security/plasma5-kwallet-pam: make it work
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228291 Bug ID: 228291 Summary: [patch] security/plasma5-kwallet-pam: make it work Product: Ports & Packages Version: Latest Hardware: Any OS: Any Status: New Severity: Affects Only Me Priority: --- Component: Individual Port(s) Assignee: k...@freebsd.org Reporter: t...@freebsd.org Assignee: k...@freebsd.org Flags: maintainer-feedback?(k...@freebsd.org) Created attachment 193457 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=193457&action=edit patch - Remove USES=gettext and add USE_KDE=wallet_run. - Patch the code so kwalletd can be found in LOCALBASE. - Replace the use of socat with netcat which is in base. Note that netcat is bidirectional. It sends input from stdin over the socket and input from the socket to stdout. After receiving EOF on stdin it keeps polling the socket for input until the other end closes the connection, which in the case of kwalletd never happens because it's also waiting for more data until the other end closes the connection. Call netcat using "nc | :" so its stdout is a pipe of which the far end closes immediately (stdout itself stays open). Without a valid stdout netcat becomes unidirectional. Using "nc >&-" doesn't work because it closes stdout itself and its descriptor will be reused for the socket so netcat thinks it has a valid stdout. - Add an upstream followup to the CVE patches committed in ports r469032. - Fix an off-by-one buffer size check. - Fix a call to bind(2). The third argument should be the size of the sockaddr struct. It contains an extra field besides sun_path and sun_family in BSD so the name of the socket got truncated. -- You are receiving this mail because: You are the assignee for the bug.