From 2b288236e80938d29324072a823f46861bd07cd3 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Thu, 3 Nov 2016 20:57:37 +0100 Subject: [PATCH] loadfont: 32k size limit is not enough Since our "read to malloced buf" routines only gradually grow allocations, let's be generous here and allow 128k. Reported by Alex Henrie Signed-off-by: Denys Vlasenko --- console-tools/loadfont.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/console-tools/loadfont.c b/console-tools/loadfont.c index 032506d6d..58073e0dc 100644 --- a/console-tools/loadfont.c +++ b/console-tools/loadfont.c @@ -319,8 +319,10 @@ int loadfont_main(int argc UNUSED_PARAM, char **argv) * We used to look at the length of the input file * with stat(); now that we accept compressed files, * just read the entire file. + * Len was 32k, but latarcyrheb-sun32.psfu is 34377 bytes + * (it has largish Unicode map). */ - len = 32*1024; // can't be larger + len = 128*1024; buffer = xmalloc_read(STDIN_FILENO, &len); // xmalloc_open_zipped_read_close(filename, &len); if (!buffer) @@ -405,7 +407,7 @@ int setfont_main(int argc UNUSED_PARAM, char **argv) } } // load font - len = 32*1024; // can't be larger + len = 128*1024; buffer = xmalloc_open_zipped_read_close(*argv, &len); if (!buffer) bb_simple_perror_msg_and_die(*argv); -- 2.25.1