de2f9a69e7
Added patch for crash if terminal is larger than 2048 bytes. Also added patch for CTRL-N/CTRL-P for next/previous element. While here, correct license format Signed-off-by: Nathan Owens <ndowens04@gmail.com>
52 lines
1.4 KiB
Diff
52 lines
1.4 KiB
Diff
From 341375179514bfd96f2d6001df15a4079631491b Mon Sep 17 00:00:00 2001
|
|
From: Nachiketa Prachanda <nchkta@gmail.com>
|
|
Date: Tue, 31 Jan 2017 12:08:48 -0800
|
|
Subject: [PATCH] out_curses: use xcalloc instead of a fixed buffer
|
|
|
|
In put_line(), replace the fixed onstack buffer with a xcalloc-ed buffer.
|
|
This fixes a bmon crash with terminal size larger than 2048 bytes. The crash
|
|
be reproduced with
|
|
$ stty cols 2100
|
|
$ bmon ....
|
|
|
|
Signed-off-by: Nachiketa Prachanda <nchkta@gmail.com>
|
|
---
|
|
src/out_curses.c | 16 +++++++++-------
|
|
1 file changed, 9 insertions(+), 7 deletions(-)
|
|
|
|
diff --git src/out_curses.c src/out_curses.c
|
|
index 6d70ae3..e5317de 100644
|
|
--- src/out_curses.c
|
|
+++ src/out_curses.c
|
|
@@ -147,22 +147,24 @@ static char *float2str(double value, int width, int prec, char *buf, size_t len)
|
|
static void put_line(const char *fmt, ...)
|
|
{
|
|
va_list args;
|
|
- char buf[2048];
|
|
+ char *buf;
|
|
+ int len;
|
|
int x, y __unused__;
|
|
|
|
- memset(buf, 0, sizeof(buf));
|
|
getyx(stdscr, y, x);
|
|
|
|
+ len = cols - x;
|
|
+ buf = xcalloc(len+1, 1);
|
|
+
|
|
va_start(args, fmt);
|
|
- vsnprintf(buf, sizeof(buf), fmt, args);
|
|
+ vsnprintf(buf, len+1, fmt, args);
|
|
va_end(args);
|
|
|
|
- if (strlen(buf) > cols-x)
|
|
- buf[cols - x] = '\0';
|
|
- else
|
|
- memset(&buf[strlen(buf)], ' ', cols - strlen(buf)-x);
|
|
+ if (strlen(buf) < len)
|
|
+ memset(&buf[strlen(buf)], ' ', len - strlen(buf));
|
|
|
|
addstr(buf);
|
|
+ xfree(buf);
|
|
}
|
|
|
|
static void center_text(const char *fmt, ...)
|