RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1484274 - There is an infinite loop in comp_scan.c of libncurses.
Summary: There is an infinite loop in comp_scan.c of libncurses.
Keywords:
Status: CLOSED DUPLICATE of bug 1488915
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: ncurses
Version: 7.5-Alt
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Miroslav Lichvar
QA Contact: qe-baseos-daemons
URL:
Whiteboard:
Depends On:
Blocks: CVE-2017-13728
TreeView+ depends on / blocked
 
Reported: 2017-08-23 07:43 UTC by owl337
Modified: 2018-07-27 15:24 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-07-27 15:24:05 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Triggered by " ./tic POC7 " (122 bytes, application/x-rar)
2017-08-23 07:43 UTC, owl337
no flags Details

Description owl337 2017-08-23 07:43:37 UTC
Created attachment 1316974 [details]
Triggered by " ./tic POC7 "

Description of problem:

There is an infinite loop in comp_scan.c of libncurses. 

Version-Release number of selected component (if applicable):

<= latest version

How reproducible:

./tic POC7

Steps to Reproduce:


$ ./tic POC7
"fuzz/info2cap_output/crashes/id:000000,sig:11,src:000041,op:havoc,rep:32", line 1, col 9, terminal 'a+du1': Illegal character (expected alphanumeric or @%&*!#) - '/'
"fuzz/info2cap_output/crashes/id:000000,sig:11,src:000041,op:havoc,rep:32", line 2, col 18, terminal 'a+du1': Illegal character (expected alphanumeric or @%&*!#) - '/'


The GDB debugging information is as follows:

(gdb) r
...
Breakpoint 2, next_char () at ../ncurses/./tinfo/comp_scan.c:209
209			if (fgets(result + used, (int) (allocated - used), yyin) != 0) {
(gdb) bt 
#0  next_char () at ../ncurses/./tinfo/comp_scan.c:209
#1  0x000000000043fa3d in _nc_get_token (silent=false) at ../ncurses/./tinfo/comp_scan.c:392
#2  0x0000000000442a20 in _nc_parse_entry (entryp=0x7fffffffaf48, literal=0, silent=false)
    at ../ncurses/./tinfo/parse_entry.c:216
#3  0x000000000043db23 in _nc_read_entry_source (fp=<optimized out>, buf=<optimized out>, literal=0, silent=false, 
    hook=0x40e230 <immedhook>) at ../ncurses/./tinfo/comp_parse.c:225
#4  0x0000000000403039 in main (argc=<optimized out>, argv=<optimized out>) at ../progs/tic.c:958
...

The vulnerability was triggered in function:
next_char () at ../ncurses/./tinfo/comp_scan.c:223
223			if ((bufptr = bufstart) != 0) {

Code:
194		do {
195		    size_t used = 0;
196		    bufstart = 0;
197		    do {
198			if (used + (LEXBUFSIZ / 4) >= allocated) {
199			    allocated += (allocated + LEXBUFSIZ);
200			    result = typeRealloc(char, allocated, result);
201			    if (result == 0)
202				return (EOF);
203			    if (bufstart)
204				bufstart = result;
205			}
206			if (used == 0)
207			    _nc_curr_file_pos = ftell(yyin);
208	
209			if (fgets(result + used, (int) (allocated - used), yyin) != 0) {
210			    bufstart = result;
211			    if (used == 0) {
212				if (_nc_curr_line == 0
213				    && IS_TIC_MAGIC(result)) {
214				    _nc_err_abort("This is a compiled terminal description, not a source");
215				}
216				_nc_curr_line++;
217				_nc_curr_col = 0;
218			    }
219			} else {
220			    if (used != 0)
221				_nc_STRCAT(result, "\n", allocated);
222			}
223			if ((bufptr = bufstart) != 0) {
224			    used = strlen(bufptr);
225			    while (iswhite(*bufptr)) {
226				if (*bufptr == '\t') {
227				    _nc_curr_col = (_nc_curr_col | 7) + 1;
228				} else {
229				    _nc_curr_col++;
230				}
231				bufptr++;
232			    }
233	
234			    /*
235			     * Treat a trailing <cr><lf> the same as a <newline> so we
236			     * can read files on OS/2, etc.
237			     */
238			    if ((len = strlen(bufptr)) > 1) {
239				if (bufptr[len - 1] == '\n'
240				    && bufptr[len - 2] == '\r') {
241				    len--;
242				    bufptr[len - 1] = '\n';
243				    bufptr[len] = '\0';
244				}
245			    }
246			} else {
247			    return (EOF);
248			}
249		    } while (bufptr[len - 1] != '\n');	/* complete a line */
250		} while (result[0] == '#');	/* ignore comments */
251	    } else if (*bufptr == '\t') {

Actual results:

crash

Expected results:

crash

Additional info:

Credits:

This vulnerability is detected by team OWL337, with our custom fuzzer collAFL. Please contact ganshuitao   and chaoz.cn if you need more info about the team, the tool or the vulnerability.

Comment 2 Thomas E. Dickey 2017-08-23 21:44:53 UTC
I made a fix which will be in the next set of updates.

Comment 3 Thomas E. Dickey 2017-08-23 22:00:14 UTC
By the way, the relevant code is in libtic, both for RHEL7 and Fedora.  You might find it helpful when reporting bugs in RHEL to use the same configuration details as their packages.


Note You need to log in before you can comment on or make changes to this bug.