LightDM Remote Session FreeRDP

Merge lp:~ted/lightdm-remote-session-freerdp/with-nla into lp:lightdm-remote-session-freerdp

  1. with-nla
  2. Merge into trunk
Proposed by Ted Gould
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 12
Merged at revision: 13
Proposed branch: lp:~ted/lightdm-remote-session-freerdp/with-nla
Merge into: lp:lightdm-remote-session-freerdp
Diff against target: 12 lines (+1/-1)
1 file modified
freerdp-session.in (+1/-1)
To merge this branch: bzr merge lp:~ted/lightdm-remote-session-freerdp/with-nla
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
jenkins (community) continuous-integration Approve
Review via email: mp+122151@code.launchpad.net

Commit message

Remove the NLA exclusion

Description of the change

Security review of the PAM FreeRDP suggested reenabling this. Doing so for the session as well.

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:12
http://jenkins.qa.ubuntu.com/job/lightdm-remote-session-freerdp-ci/3/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/lightdm-remote-session-freerdp-ci/./label=quantal/3/console

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Be secure :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'freerdp-session.in'
2--- freerdp-session.in 2012-08-21 19:34:12 +0000
3+++ freerdp-session.in 2012-08-30 21:13:25 +0000
4@@ -1,7 +1,7 @@
5 #!/bin/bash
6
7 if [ -e ~/.freerdp-socket ]; then
8- @pkglibexecdir@/socket-sucker | /usr/bin/xfreerdp --plugin rdpsnd.so --no-nla -f --from-stdin
9+ @pkglibexecdir@/socket-sucker | /usr/bin/xfreerdp --plugin rdpsnd.so -f --from-stdin
10 else
11 zenity --warning --text="Unable to locate FreeRDP socket"
12 fi;

Subscribers

People subscribed via source and target branches

to all changes: