Issue 15 in openssh-lpk: LPK patch updated for OpenSSH 6.2p1

[codesite...@google.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 15 by carsten....@evalcode.net: LPK patch updated for OpenSSH
6.2p1
http://code.google.com/p/openssh-lpk/issues/detail?id=15

I have updated and attached the patch for OpenSSH 6.2p1 - though could use
a little refactoring since OpenSSH refactored some related methods and I
tried to not actually change the patch for now.

I am currently using it with Debian 6.0.3 (OpenSSL 0.9.8) and Ubuntu 12.04
LTS (OpenSSL 1.0.1).


Attachments:
openssh-lpk-6.2p1-0.3.9.patch 21.4 KB

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[codesite...@google.com]

Comment #1 on issue 15 by bgmi...@gmail.com: LPK patch updated for OpenSSH
6.2p1
http://code.google.com/p/openssh-lpk/issues/detail?id=15

I believe the reason the related methods were refactored was the inclusion
of a feature that now obsoletes this patch:

20121030
- (djm) OpenBSD CVS Sync
- mar...@cvs.openbsd.org 2012/10/05 12:34:39
[sftp.c]
fix signed vs unsigned warning; feedback & ok: djm@
- d...@cvs.openbsd.org 2012/10/30 21:29:55
[auth-rsa.c auth.c auth.h auth2-pubkey.c servconf.c servconf.h]
[sshd.c sshd_config sshd_config.5]
new sshd_config option AuthorizedKeysCommand to support fetching
authorized_keys from a command in addition to (or instead of) from
the filesystem. The command is run as the target server user unless
another specified via a new AuthorizedKeysCommandUser option.

patch originally by jchadima AT redhat.com, reworked by me; feedback
and ok markus@

There is an implementation of the AuthorizedKeysCommand for fetching keys
from LDAP. For example, on RHEL, this is available in the openssh-ldap
package.

[codesite...@google.com]

Comment #2 on issue 15 by carsten....@evalcode.net: LPK patch updated for
OpenSSH 6.2p1
http://code.google.com/p/openssh-lpk/issues/detail?id=15

Ok thats nice to know now :) Definitely to prefer since I can stick then to
standard packages again.

Anyway there might still be folks that would prefer a built-in solution
(like me actually, but rather built-in by default).

Btw. just noticed that the patch did not include new files / was incomplete
- here a fixed one.

Attachments:
openssh-lpk-6.2p1-0.3.9.patch 80.8 KB

[codesite...@google.com]

Comment #3 on issue 15 by alien...@gmail.com: LPK patch updated for OpenSSH
6.2p1
http://code.google.com/p/openssh-lpk/issues/detail?id=15

Don't I see the misspelled lines in the latest patch?
The lines to remove are marked with "**" vs. "---", and lines to add are
marked with "---" vs. "+++". Pls verify.

[codesite...@google.com]

Comment #4 on issue 15 by alien...@gmail.com: LPK patch updated for OpenSSH
6.2p1
http://code.google.com/p/openssh-lpk/issues/detail?id=15

... and in addition to file name marks, the lines to add are also marked
with "-" vs. "+", and the same "opposite" is for the deleted lines. Pls
clarify.