[PATCH] Screen buffers ESC keypresses indefinitely since sgr support

bug #57748

sync man page

autogen.sh: add --install option for autoreconf

exit_with_usage() returns 1 only in case of error

"screen -h|--help" should NOT return 1

Sync with screen-v4: 7908c33f738d5dfc19adfdcf32bb9407edef71a4

Fix call to undeclared function chmod

The chmod function comes from the header file sys/stat.h
which was not included in the source file display.c,
and thus the error.
This patch should fix the build error.

Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>

Fix bashisms in configure.ac

bug #62076

autoupdate AC_PREREQ 2.71

Missing signal sending permission check on failed query messages

When run as setuid root, one can send a query message to the
privileged screen process via its unix socket in order to force
it to send SIGHUP to a PID that can be freely specified in the
query packet.
Processes that do not explicitly handle SIGHUP will simply terminate.

Signed-off-by: Alexander Naumov <alexander_naumov@opensuse.org>

httpss->https

http => https

http => https

Avoid zombies after shell exit

As documented in libutempter:
"During execution of the privileged process spawned by these
functions, SIGCHLD signal handler will be temporarily set to
the default action."

Thus in case a SIGCHLD has been lost, we send a SIGCHLD to
oneself in order to avoid zombies:
https://savannah.gnu.org/bugs/?25089

bugfix CVE-2021-26937

It allows remote attackers to cause a denial of service
(invalid write access and application crash) or possibly
have unspecified other impact via a crafted UTF-8
character sequence.

bugfix: https://savannah.gnu.org/bugs/?60030

Signed-off-by: Alexander Naumov <alexander_naumov@opensuse.org>

man page: copyright update

[PATCH 2/2] trivial: docs: Add missing ')'.

A cross-reference was intended to be placed inside parentheses,
but the closing parenthesis was forgotten; this commit inserts
that missing closing parenthesis.

[PATCH 1/2] trivial: docs: Fix typo (s/paramter/parameter)

In the description of the 'stuff' command, there
was the typo "paramter"; this commit makes that
word "parameter".

bugfix: option -X ignores specified user in multiuser env

bug #37437

Thanks to Taj Morton

TERMCAP_BUF is used in place of TERMCAP_BUFSIZE

Expand d_xtermosc array in struct Display

Commit 7059bff expands index range of "typ2" by one without expanding
affected arrays. d_xtermosc in struct display is one of these.

Related: 7059bff20a28778f9d3acf81cad07b1388d02309
  (ansi: add support for xterm OSC 11)
Related: eb2be1adf92d58bd8f4ca3458eb04da38bf33c2b
  (Fix out of bounds access when setting w_xtermosc after OSC 49)

Signed-off-by: Václav Doležal <vdolezal@redhat.com>

Increase permitted length of OSC

hyperlink feature used by some terminals requires lots of characters
https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda#length-limits
mentions around 2083 characters, set it to a bit more.

Bug: 57718

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix out of bounds access when setting w_xtermosc after OSC 49

echo -e "\e]49\e;                                    \n\ec"
crashes screen.

This happens because 49 is divided by 10 and used as table index
resulting in access to w_xtermosc[4], which is out of bounds with table
itself being size 4. Increase size of table by 1 to 5, which is enough
for all current uses.

As this overwrites memory based on user input it is potential security
issue.

Reported-by: pippin@gimp.org
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Add escape to show number of currently open windows in hardstatus

String escape format of %C for number of windows with - prefix for
group window count.

Bug: 14484

Signed-off-by: Ed Neville <ed@s5h.net>

Fix 100% cpu use on one core with poll

If there is no event timeout set just wait for 1 second instead of
setting poll timeout to 0.

Bug: 57697

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Haiku port: Check for some libraries

* libnetwork for sockets,
* libbsd for openpty.

fix mismatched types in prototype

not everyone defines pid_t as int...

remove hardcoded -lutil

AC_SEARCH_LIBS prepends the found library already.

add dirent.h test back to configure

Haiku doesn't have sys/dir.h which is deprecated anyway,
so it's better to actually use dirent.h if found.

build: acls.c: mark unused parameters

* src/acls.c (DoSu): avoid four unused-variable warnings

build: tests/test-winmsgbuf.c: avoid GCC 10 alloca warning

* src/tests/test-winmsgbuf.c (main): Replace alloca with malloc/free
to prevent the following warning: tests/test-winmsgbuf.c:298:19:\
  warning: implicit declaration of function ‘alloca’; did you mean \
  ‘calloc’?

build: test-winmsgbuf.c: add const to avoid GCC 10 warning

* src/tests/test-winmsgbuf.c:45: make char * const
This avoids the following warning:
tests/test-winmsgbuf.c:45:17: warning: initialization of\
‘char *[...]'from incompatible pointer type ‘const char *[...]'

cleanup: acls.c: move unused variables into #if-0 block

* src/acls.c (DoSu): avoid two unused-variable warnings

build: cast NULL to (char *) to prevent GCC 10 warning

* src/acls.c (GrowBitfield): replace NULL by (char *)NULL.
This avoids the following warning:
acls.h:44:35: warning: dereferencing ‘void *’ pointer

Refactor condition handling to use offset instead of pointer

If wmc_else() is called after wmb_expand() it can overwrite pointer to
old value. To avoid this issue refactor condition code to use offset
into parsed string instead.

Reported-by: pippin@gimp.org
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Copyright update for 2020

Fix return code of '--version' and '-v'

Bug #57571

Fix code formatting

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix variable type

ParseAttrColor returns uint64_t and ApplyAttrColor takes uint64_t as
argument, int was used instead, which may cause vertical bar to not have
color applied on split.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Disable exclusive mode on TTY device when closing

When opening a TTY we enable exclusive mode to prevent other tools
messing with our connection. When we are done using the TTY the
exclusive mode must be disabled so the TTY can be reconnected to
later. We remember to do this when a break is sent to the device,
but not on window close.

This change ensures we disable exclusive mode on the TTY device
whenever the window is closed.

bug #52248

Create TERMCAP entries limited to 1023 bytes by default.

TERMCAP_BUF defaults to 1023 to create TERMCAP entries that work on
most systems.  To save space, TERMCAP is unwrapped, and vt220 extra
keys are skipped (unless TERMCAP_BUF > 1023); navigation keys are
still included.  Entries larger than TERMCAP_BUF are now truncated,
and no longer Panic screen.

Termcap entries are still wrapped when saved to a file.

Signed-off-by: Scott Shambarger <devel@shambarger.net>

Fix coredump on xterm and rxvt

Here's the rub: with TERM=xterm (or rxvt, for that matter), Km
("key_mouse", "Mouse event has occured") is not set (and therefore
NULL), but InitTermcap() (termcap.c:230) happily tries to strdup() that,
which gets us that segfault.

As a band-aid, catch that NULL and don't strdup().

Signed-off-by: Marcin Cieślak <saper@saper.info>

Count timeout in milliseconds instead of using struct timeval

poll() accepts timeout in milliseconds, so there is no need to keep
timeout in struct timeval.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Convert select() to poll() in sched.c

select() limits number of file descriptors that can be used by screen.
Migrate to poll() to avoid this limitation.

As can be seen in case of scheduler it requires quite some changes, care
must be taken to count poll() events properly.

Bug: 55697

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Convert select() to poll() in screen.c

select() limits number of file descriptors that can be used by screen.
Migrate to poll() to avoid this limitation.

Bug: 55697

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Convert select() to poll() in display.c

select() limits number of file descriptors that can be used by screen.
Migrate to poll() to avoid this limitation.

Bug: 55697

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Make closeallfiles() faster

Optimize startup time, making closeallfiles() faster, by doing less
system calls. Instead of calling close for each possible file, use
poll() to check if file exist at all. On linux with open file limit set
to 1048576, it should do 1024 poll() calls instead of 1048576 close().

Bug: 55618

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Using 'https' instead of 'http'

Update ambiguous and wide characters tables to Unicode 12.1.0

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Get rid of externs in telnet.c

Just include correct headers.
We need to expose af properly in screen.h for this to work.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix broken mouse after ncurses 6.1

ncurses 6.1 changed kmous capability from "\e[M" to "\e[<". It seems to
be done to signal that terminal supports sgr mouse mode. screen assumed
that if kmous is set to "\e[M" it is on xterm compatible terminal
anyway, so just dynamically detect which one is used and override
relevant kmapdef.

InitKeytab() is moved, so kmapdef[] can be overriden before
initialization, as InitTermcap() needs to run first, as far as I can
tell this should have no consequences.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix some gcc-9 warning in utmp

Shouldn't have really used strncpy, for something that is effectively
binary data.

Also cleanup structs when defining instead of adding memsets everywhere.

Revert "Remove unnecessary condition check"

This reverts commit ceb9b1f6ec6378c83566af0029ddefcc825ba6d4.

I need to look at this again, there is a report that commit in question
may cause problems and apparently logic may have failed me when I
removed this code, as it probably should stay and everything after it is
unreacheable.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Clean whole buffer if we don't want to parse it

Fixes problem when pressing arrows in some prompts causes arrows to stop
in other places.

For example pressing up arrow in 'kill --confirm' prompt stops up arrow
from working on 'windowlist'

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Remove unnecessary condition check

According to parent "if" condition this is never the case

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Close file descriptor after query command

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix -L option when passed to backend via Msg

Breaks backward compatibility

This fixes a problem when user runs screen with "-L" option in already
existing screen. We need to pass information about theflag to backend.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Define data pointer in Event as void

We don't know format of data we pass around, so we may as well define
pointer as void *. Gets rid of -Wcast-align warning with clang.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix D_processinputdata type

Define d_processinputdata as struct pwdata instead of doing casts back
and forth. Removes clang warning with -Wcast-align.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Copyright update for 2019

Typo in man page

bug #56027

fix UTF-8 characters with more than 2 bytes

Characters are stored as uint32 nowadays. Looking at the lower bytes
only seems to be an oversight from the fontx clean up.

Fixes: f18f5d0da1d4 get rid of fontx

drop unnecessary BEL from bracketed paste mode sequence

there is no BEL in enable/disable bracketed paste mode sequence, so we
shouldn't unnecessarily print it

Bug: #55709

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix extra characters when sending a command in one go

This closes bug #52372: Giberish printed to screen window in certain
situations.

When sending "^An", most of the time, screen first receives "^A", then
it receives the "n".  This means that in the ProcessInput2 function,
ilen == 1 after the escape character has been seen, ilen is decremented
and the ProcessInput2 function returns.

When "^An" is sent in one go, for example when pasting or when using a
special shortcut, ilen is not 1 when the escape character is seen.  In
this case, the s variable was incremented, but ilen was not decremented.
This leads the function to read an extra character which does not
exists.

This regression was introduced in 2fab4d6f73, as reported in the bug
report.

This can be tested by having a command in the selection:

echo -ne '\x01n' | xclip -i

And pasting it (in this case, you might see the closing bracketed paste
sequence, but that is another problem).

Bug: #52372
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

make build date reproducible

Based on https://reproducible-builds.org/docs/source-date-epoch/
They suggest "BUILD_DATE ?=", but it seems to be evaluated for each file
separately, so us "BUILD_DATE :=" to set it once.

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix blanker to work when screen is suid root

* Change RunBlanker to call OpenDevice so permissions on slave
  PTY are correctly set.
* Update handling of file descriptors after fork to be similar to
  ForkWindow on at pty (fixes debug and leaked descriptors)
* Allow display of error message when display blocked by blanker
  (because message is probably from blanker failing to start)

Bug: 55512

Cherry-picked form screen-v4, with some modifications

Signed-off-by: Scott Shambarger <devel@shambarger.net>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Prevent Panic causing Panic, and children removing sockets

* Set eff_uid/eff_gid after setuid/setgid to prevent nested Panic
  MakeClientSocket calls xseteuid(eff_uid=0) - results in nested
  Panic and SendErrorMsg not getting sent.
* Set ServerSocket to -1 after fork so that child Panic doesn't
  remove socket in eexit.

Bug: 55511

Cherry-picked from screen-v4 with some modifications

Signed-off-by: Scott Shambarger <devel@shambarger.net>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

mark AttacherFinit, AttacherFinitBye & Attacher as noreturn

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

mark backticks as static

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

fix flow of 'at' command

we just returned without freeing 's' and resetting EffectiveAclUser,
which per comment at the beginning should be reset

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

sched() can be marked as noreturn

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

remove unneeded cast

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

RunBlanker prototype should be in display.h

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

wmb_create takes no parameters

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

in verbose command, we can ignore return value from ParseOnOff

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

in log command toggle log only if we successfully parsed argument

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

in truecolor command trigger redraw only if passed correct argument

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

don't attempt to swap window with itself

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

get rid of Wformat-overflow warning

60 characters is plenty enough for reason string

window.c: In function ‘WindowDied’:
window.c:1941:33: warning: ‘%s’ directive writing up to 99 bytes into a region of size 86 [-Wformat-overflow=]
   sprintf(buf, "\n\r=== Command %s (%s) ===", reason, s ? s : "?");
                                 ^~            ~~~~~~
window.c:1941:16: note: assuming directive output of 1 byte
   sprintf(buf, "\n\r=== Command %s (%s) ===", reason, s ? s : "?");
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/stdio.h:862,
                 from logfile.h:33,
                 from window.h:39,
                 from window.c:34:
/usr/include/bits/stdio2.h:33:10: note: ‘__builtin___sprintf_chk’ output 22 or more bytes (assuming 122) into a destination of size 100
   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       __bos (__s), __fmt, __va_arg_pack ());
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

fix zombie command

commit f12ada81929f78412814a30d9c802d8633db4e27 (refactor SwapWindows)
accidentally changed killit variable causing code to never execute

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

check if fseek & ftell succeeded

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

w_tty is an array, so we don't need to perform NULL check on it

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

fix truecolor escape

it's still a hack, because to do proper detection we would need to
migrate to terminfo functions instead of termcap

Bug: 52667
Bug: 54989

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

added documentation for traversal to parent group

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

added function and key binding for traversing to parent

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

add persistent bits to window group display options

Windowlist has two toggles that affect its output: the list can be sorted in
either index or MRU order, and the list can contain either just immediate
children windows of all descendant windows.  Since window groups use the same
code for their own output, they also support these toggles.

However, in the current code base, these toggles are reset to index ordering and
display of immediate children only every time a particular window group is
swapped out of and back into a pane.

Amadeusz:
added commit message + changed variable names + some reformatting

Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>

Fix for nomem handling in resize.c:ChangeWindowSize()

Move `nomem' label of ChangeWindowSize() to the end of function and
add test for value of `nhlines'

Signed-off-by: Vaclav Dolezal <vdolezal@redhat.com>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net

Revert "those 0 assignment made rest of code totally not working"

This reverts commit ff98d7ff5847e07a55b0c40c2ccc3bc430226ca0.

This can potentially cause double-free. `nmlines' and `nhlines'
should be freed in preceding lines.

`nmlines' and `nhlines' are not used in the rest of the function
except in `nomem' label and their valuse are copied into `p', so
their value should be zeroed.

Signed-off-by: Vaclav Dolezal <vdolezal@redhat.com>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net

Fix file descriptor leak

Signed-off-by: Vaclav Dolezal <vdolezal@redhat.com>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net

Use memcpy(3) in string substitution

Signed-off-by: Vaclav Dolezal <vdolezal@redhat.com>
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net

apply patch by Therese fixing some doc issues

This fixes some issues when documentation is generated for online
viewing.

bugfix: crash on dinfo, "stack smashing detected"

This seems to be caused by faulty pointer/buffer-length
tracking. Pointer position 'p', meant to be incremented
by the length of the string written to the buffer, is
instead incremented by (length of the buffer - length of
the string written to the buffer), quickly shooting past
the end of the allocated buffer space.

doc: fix typos

Hi there,

just found a few typos, see attachment.

(please, CC me)

Thanks.

--
https://www.juef.space/

From 745ba353867142d3e00f4d2ab06962ba0d3aaaab Mon Sep 17 00:00:00 2001
From: Svyatoslav Mishyn <svyatoslav.mishyn@gmail.com>
Date: Wed, 4 Jul 2018 21:40:48 +0300
Subject: [PATCH] doc: fix typos

add %T escape to show current tty for window

remove character substitution

it may be not compatible with some man page viewers

fix manpage

manpage: Explain window permissions in a table

manpage: Use monospaced font to render example of "displays"

manpage: Indent both examples for readreg

manpage: keep nonblock and status parameters on the same line

manpage: explain flow control state indicators with a table

manpage: Remove horizontal lines on large tables

Remove "allbox" tbl option from large keybinding table.
When formatting for print, large tbl tables need to be handled
specially to split nicely across the pages; instead we
just add horizontal lines and we let them overflow the pages.

The input translation table seems to fit nicely on one page
and is left as is with the "allbox" option.