git: 9front

Download patch

ref: 86ce774a9d80071ac66a850176283d29d59e9dd4
parent: a3c13e004ecde7334ab2088111c7bb051910fa3c
author: cinap_lenrek <cinap_lenrek@felloff.net>
date: Sun Oct 31 08:39:46 EDT 2021

libc: fix overflow of domain component rune buffer for idn2utf()

If the source string has a run of more than 256 runes without
a "." dot, we'd overflow the runebuffer in idn2utf().

The utf2idn() routine had a check in the while loop, but that
is actually wrong too, as it would insert a dot and restart
the loop in the middle of a domain component. Just error
out if a domain component is too long.

--- a/sys/src/libc/9sys/idn.c

+++ b/sys/src/libc/9sys/idn.c

@@ -200,6 +200,8 @@

 			n = chartorune(&r, cp+nc);

 			if(r == '.')

 				break;

+			if(nr >= nelem(rb))

+				return -1;

 			rb[nr++] = r;

 			nc += n;

 		}

@@ -234,10 +236,12 @@

 	cp = name;

 	for(;;){

 		nc = nr = 0;

-		while(cp[nc] != 0 && nr < nelem(rb)){

+		while(cp[nc] != 0){

 			n = chartorune(&r, cp+nc);

 			if(r == '.')

 				break;

+			if(nr >= nelem(rb))

+				return -1;

 			rb[nr++] = r;

 			nc += n;

 		}

-- 

⑨