ringtools/imy: fix duplicate definition of tdma_durations[] The bug was reported by Vadim Yanitskiy <fixeria@osmocom.org>, although the present fix is slightly different from the contributed patch: because main.c doesn't need this tdma_durations[] array at all, let's simply remove the reference to this array from main.c rather than turn it into an extern. I no longer remember my original thought flow that resulted (by mistake) in tdma_durations[] being multiply defined in main.c and durations.c. My intent might have been to define all globals in main.c and have the reference in durations.c be an extern - and I missed that extern - but without clear memory, I have no certainty. In any case, having this data array defined in the same module that fills it (durations.c) is sensible, so let's make it the new way.

/*
 * This library module implements the function for converting UTF-8 input
 * to UCS-2 in outgoing SMS composition.
 */

#include <sys/types.h>

utf8_to_ucs2(inbuf, outbuf, outmax, outlenp)
	u_char *inbuf;
	u_short *outbuf;
	unsigned outmax, *outlenp;
{
	u_char *ip = inbuf;
	u_short *op = outbuf;
	unsigned outcnt = 0, c, n, uni;

	while (c = *ip++) {
		if (c < 0x80) {
			uni = c;
			goto gotuni;
		}
		if (c < 0xC0 || c > 0xEF)
			return(-1);
		uni = c & 0x1F;
		if (c >= 0xE0)
			n = 2;
		else
			n = 1;
		for (; n; n--) {
			c = *ip++;
			if (c < 0x80 || c > 0xBF)
				return(-1);
			uni <<= 6;
			uni |= c & 0x3F;
		}
gotuni:		if (outcnt >= outmax)
			return(-2);
		*op++ = uni;
		outcnt++;
	}
	*outlenp = outcnt;
	return(0);
}