Package: screen
Version: 4.1.0~20120320gitdb59704-7
Severity: normal
Tags: patch

Dear Maintainer,

when attaching to a screen that is owned by a different user with a long
name (> 20 characters), I get the following error:

host ~ # screen -x this-is-a-very-long-username/
Screen owner name too long - sorry.

screen originally had a limit of 20 characters for usernames and
wouldn't work at all if your username was longer, but this was mostly
fixed in a Debian patch. Apparently the patch overlooked this specific
situation so that this issue still occurs.

I've attached a patch that resolved this issue for me.


-- System Information:
Debian Release: 7.3
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: i386 (x86_64)
Foreign Architectures: amd64

Kernel: Linux 3.10-0.bpo.3-amd64 (SMP w/8 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages screen depends on:
ii  dpkg          1.16.12
ii  install-info  4.13a.dfsg.1-10
ii  libc6         2.13-38
ii  libncurses5   5.9-10
ii  libpam0g      1.1.3-7.1
ii  libtinfo5     5.9-10

screen recommends no packages.

Versions of packages screen suggests:
pn  iselect | screenie | byobu  <none>

-- Configuration Files:
/etc/screenrc changed [not included]

-- debconf information excluded
Description: fix "Screen owner name too long - sorry" error
 The patch 49long-usernames.patch increases the maximum user name length that
 screen supports from 20 to 50 characters. Unfortunately, the limit still
 applies when trying to attach to a multiuser screen:
 .
 host ~ # screen -x veeeeeeeeeeeeerylongusername/
 Screen owner name too long - sorry.
 .
 This patch raises the length limit for owner names too.
 .
 screen (4.1.0~20120320gitdb59704-7) unstable; urgency=low
 .
   * Extend 60-644788-screen-4.1.0-4.0.3-interoperability.patch:
     + Add support for detaching (Closes: #684342)
     + Document remaining issues in debian/NEWS
Author: Axel Beckert <a...@debian.org>
Bug-Debian: http://bugs.debian.org/684342

---
The information above should follow the Patch Tagging Guidelines, please
checkout http://dep.debian.net/deps/dep3/ to learn about the format. Here
are templates for supplementary fields that you might want to add:

Author: Martin von Wittich <martin.von.witt...@iserv.eu>
Last-Update: 2014-01-16

--- screen-4.1.0~20120320gitdb59704.orig/screen.c
+++ screen-4.1.0~20120320gitdb59704/screen.c
@@ -1000,7 +1000,7 @@ char **av;
   if (strlen(LoginName) > MAX_USERNAME_LEN)
     Panic(0, "LoginName too long - sorry.");
 #ifdef MULTIUSER
-  if (multi && strlen(multi) > 20)
+  if (multi && strlen(multi) > MAX_USERNAME_LEN)
     Panic(0, "Screen owner name too long - sorry.");
 #endif
   if (strlen(home) > MAXPATHLEN - 25)

Reply via email to