systemd-fsckd: useless CR displayed on console
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| systemd (Ubuntu) |
Undecided
|
Dimitri John Ledkov |
Bug Description
Hello,
My system is "Ubuntu 16.04.2 LTS" (grep DISTRIB_DESCRIPTION /etc/lsb-release).
The systemd package is installed:
$ dpkg -l | grep systemd
> systemd 229-4ubuntu17
I noticed that approximately 15 seconds after the boot, a useless
Carriage Return ('\r') is displayed on the console (/dev/console).
This character interacts with the display of the user currently logued in
on the console. This screws the current line, when users is pressing keys.
After monitoring system activity, I found the culprit:
/lib/
To test, run:
service systemd-fsckd stop # In case, it is still running.
service systemd-fsckd start
Go to the console, wait for approximately 15 seconds, and you see the cursor
jumping from its current position to the beginning of the line. This is the BUG.
Note: the console is not the X11 terminal (xterm, lxterminal, etc.) ; the
console is the text tty reached with Control-Alt-F1.
The systemd-
/lib/
This is the daemon which displays two useless '\r' characters.
I went to:
http://
I downloaded:
http://
The file debian/
contains:
+static int manager_
[...]
+ if (message) {
+ fprintf(console, "\r%s\r%n", message, &l);
+ if (m->clear < (size_t)l)
+ m->clear = (size_t)l;
+ } else {
+ fputc('\r', console);
+ for (j = 0; j < m->clear; j++)
+ fputc(' ', console);
+ fputc('\r', console);
+ }
So, when no message was previously displayed, "m->clear" is still set to 0.
Then, when the program ends it calls:
+ /* clear last line */
+ manager_
However, in the "else" above, two '\r' characters are displayed, surrounding
no space characters.
So, when no message was previously displayed, there is nothing to clear, so
no '\r' character to display.
A trivial patch is attached. It is untested.
Laurent (laurent35263) wrote : | #1 |
Launchpad Janitor (janitor) wrote : | #3 |
Status changed to 'Confirmed' because the bug affects multiple users.
Changed in systemd (Ubuntu): | |
status: | New → Confirmed |
Dimitri John Ledkov (xnox) wrote : | #4 |
Interesting.
Dimitri John Ledkov (xnox) wrote : | #5 |
Didrocks, would you be able to review this patch?
Changed in systemd (Ubuntu): | |
assignee: | nobody → Didier Roche (didrocks) |
Didier Roche (didrocks) wrote : | #6 |
Thanks for submitting this patch and sorry for the time to review it!
The patch looks good to me, I would love though to get that directly upstream rather than us carrying a distro-patch for a small issue like this, do you mind doing a PR there? https:/
Changed in systemd (Ubuntu): | |
status: | Confirmed → Incomplete |
Laurent (laurent35263) wrote : | #7 |
Hello,
Thank you for your reply.
Unfortunately, the bug is NOT in upstream systemd, but in a Ubuntu specific patch.
A few things changed since last year:
Systemd version is now 229-4ubuntu21.2.
Its source is now at:
http://
The bugged function manager_
debian/
The new patch is attached. This patch can be used to patch the Ubuntu specific patch.
Thank you.
Didier Roche (didrocks) wrote : | #8 |
Interesting, I was sure we upstreamed completely systemd-fsckd. Anyway, +1 for me, let me assign to xnox who is doing most of systemd uploads nowdays so that he attaches it to next upload.
Many thanks!
Changed in systemd (Ubuntu): | |
assignee: | Didier Roche (didrocks) → Dimitri John Ledkov (xnox) |
status: | Incomplete → Triaged |
Zhang Youfu (zhangyoufu) wrote : | #9 |
This bug still exists in 19.04 Disco. Please fix it.
tags: | added: ddstreet |
Launchpad Janitor (janitor) wrote : | #10 |
This bug was fixed in the package systemd - 245.6-2ubuntu1
---------------
systemd (245.6-2ubuntu1) groovy; urgency=medium
* Merge to Ubuntu from Debian unstable
- Dropped changes:
* dhclient-
* hwdb: Mask rfkill event from intel-hid on HP platforms (LP: #1883846)
File: debian/
https:/
* debian/
File: debian/
https:/
* debian/
(LP: #1886430)
File: debian/
https:/
-- Balint Reczey <email address hidden> Mon, 06 Jul 2020 16:30:25 +0200
Changed in systemd (Ubuntu): | |
status: | Triaged → Fix Released |
bhs (bharath-vegito) wrote : | #11 |
Requesting patch to be included in bionic-updates as well. Current systemd-version = 237-3ubuntu10.41 (https:/
summary: |
- systemd-fsckd : useless CR displayed on console + systemd-fsckd: useless CR displayed on console |
The attachment "patch_ to_remove_ useless_ cr.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]