8e3b51d51a
- fix #882305 - agetty: unstable /dev/tty* permissions - fix #885314 - hexdump segfault - fix #896447 - No newlines in piped "cal" command - fix libblkid cache usage (upstream patch) - fix #905008 - uuidd: /usr/sbin/uuidd has incorrect file permissions Signed-off-by: Karel Zak <kzak@redhat.com>
140 lines
4.1 KiB
Diff
140 lines
4.1 KiB
Diff
From c265cc40a8ece740084fbfb3ebd4c1894b3b29b7 Mon Sep 17 00:00:00 2001
|
|
From: Karel Zak <kzak@redhat.com>
|
|
Date: Thu, 17 Jan 2013 18:48:16 +0100
|
|
Subject: [PATCH 3/3] cal: don't mix ncurses output functions and printf()
|
|
|
|
Fedora 18 (ncurses-libs-5.9-7.20121017, glibc-2.16-28):
|
|
|
|
$ cal | cat
|
|
Actual results:
|
|
January 2013 Su Mo Tu We Th Fr Sa 1 2 3 4 5 6 7 8
|
|
9 10 11 1213 14 15 16 17 18 1920 21 22 23 24 25 2627 28 29 30 31
|
|
|
|
Not sure where is the problem, but it's too fragile to mix two
|
|
output methods in one code.
|
|
|
|
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=896447
|
|
Signed-off-by: Karel Zak <kzak@redhat.com>
|
|
---
|
|
misc-utils/cal.c | 43 +++++++++++++++++++++++++++----------------
|
|
1 file changed, 27 insertions(+), 16 deletions(-)
|
|
|
|
diff --git a/misc-utils/cal.c b/misc-utils/cal.c
|
|
index 60e85df..0e204b9 100644
|
|
--- a/misc-utils/cal.c
|
|
+++ b/misc-utils/cal.c
|
|
@@ -498,7 +498,7 @@ monthly(int day, int month, int year) {
|
|
do_monthly(day, month, year, &out);
|
|
for (i = 0; i < FMT_ST_LINES; i++) {
|
|
my_putstring(out.s[i]);
|
|
- putchar('\n');
|
|
+ my_putstring("\n");
|
|
}
|
|
}
|
|
|
|
@@ -533,8 +533,11 @@ monthly3(int day, int month, int year) {
|
|
do_monthly(day, next_month, next_year, &out_next);
|
|
|
|
width = (julian ? J_WEEK_LEN : WEEK_LEN) -1;
|
|
- for (i = 0; i < 2; i++)
|
|
- printf("%s %s %s\n", out_prev.s[i], out_curm.s[i], out_next.s[i]);
|
|
+ for (i = 0; i < 2; i++) {
|
|
+ snprintf(lineout, sizeof(lineout),
|
|
+ "%s %s %s\n", out_prev.s[i], out_curm.s[i], out_next.s[i]);
|
|
+ my_putstring(lineout);
|
|
+ }
|
|
for (i = 2; i < FMT_ST_LINES; i++) {
|
|
int w1, w2, w3;
|
|
w1 = w2 = w3 = width;
|
|
@@ -562,7 +565,7 @@ j_yearly(int day, int year) {
|
|
|
|
snprintf(lineout, sizeof(lineout), "%d", year);
|
|
center(lineout, J_WEEK_LEN*2 + J_HEAD_SEP - 1, 0);
|
|
- printf("\n\n");
|
|
+ my_putstring("\n\n");
|
|
|
|
for (i = 0; i < 12; i++)
|
|
day_array(day, i + 1, year, days[i]);
|
|
@@ -571,8 +574,10 @@ j_yearly(int day, int year) {
|
|
for (month = 0; month < 12; month += 2) {
|
|
center(full_month[month], J_WEEK_LEN-1, J_HEAD_SEP+1);
|
|
center(full_month[month + 1], J_WEEK_LEN-1, 0);
|
|
- printf("\n%s%*s %s\n", j_day_headings, J_HEAD_SEP, "",
|
|
- j_day_headings);
|
|
+ snprintf(lineout, sizeof(lineout),
|
|
+ "\n%s%*s %s\n", j_day_headings, J_HEAD_SEP, "",
|
|
+ j_day_headings);
|
|
+ my_putstring(lineout);
|
|
for (row = 0; row < 6; row++) {
|
|
p = lineout;
|
|
for (which_cal = 0; which_cal < 2; which_cal++) {
|
|
@@ -584,10 +589,10 @@ j_yearly(int day, int year) {
|
|
*p = '\0';
|
|
trim_trailing_spaces(lineout);
|
|
my_putstring(lineout);
|
|
- putchar('\n');
|
|
+ my_putstring("\n");
|
|
}
|
|
}
|
|
- printf("\n");
|
|
+ my_putstring("\n");
|
|
}
|
|
|
|
void
|
|
@@ -598,7 +603,7 @@ yearly(int day, int year) {
|
|
|
|
snprintf(lineout, sizeof(lineout), "%d", year);
|
|
center(lineout, WEEK_LEN*3 + HEAD_SEP*2 - 1, 0);
|
|
- printf("\n\n");
|
|
+ my_putstring("\n\n");
|
|
|
|
for (i = 0; i < 12; i++)
|
|
day_array(day, i + 1, year, days[i]);
|
|
@@ -608,8 +613,10 @@ yearly(int day, int year) {
|
|
center(full_month[month], WEEK_LEN-1, HEAD_SEP+1);
|
|
center(full_month[month + 1], WEEK_LEN-1, HEAD_SEP+1);
|
|
center(full_month[month + 2], WEEK_LEN-1, 0);
|
|
- printf("\n%s%*s %s%*s %s\n", day_headings, HEAD_SEP,
|
|
- "", day_headings, HEAD_SEP, "", day_headings);
|
|
+ snprintf(lineout, sizeof(lineout),
|
|
+ "\n%s%*s %s%*s %s\n", day_headings, HEAD_SEP,
|
|
+ "", day_headings, HEAD_SEP, "", day_headings);
|
|
+ my_putstring(lineout);
|
|
for (row = 0; row < 6; row++) {
|
|
p = lineout;
|
|
for (which_cal = 0; which_cal < 3; which_cal++) {
|
|
@@ -621,10 +628,10 @@ yearly(int day, int year) {
|
|
*p = '\0';
|
|
trim_trailing_spaces(lineout);
|
|
my_putstring(lineout);
|
|
- putchar('\n');
|
|
+ my_putstring("\n");
|
|
}
|
|
}
|
|
- putchar('\n');
|
|
+ my_putstring("\n");
|
|
}
|
|
|
|
/*
|
|
@@ -773,10 +780,14 @@ void
|
|
center(const char *str, size_t len, int separate)
|
|
{
|
|
char lineout[FMT_ST_CHARS];
|
|
+
|
|
center_str(str, lineout, ARRAY_SIZE(lineout), len);
|
|
- fputs(lineout, stdout);
|
|
- if (separate)
|
|
- printf("%*s", separate, "");
|
|
+ my_putstring(lineout);
|
|
+
|
|
+ if (separate) {
|
|
+ snprintf(lineout, sizeof(lineout), "%*s", separate, "");
|
|
+ my_putstring(lineout);
|
|
+ }
|
|
}
|
|
|
|
static void __attribute__ ((__noreturn__)) usage(FILE * out)
|
|
--
|
|
1.8.1
|
|
|