view target-utils/libtiffs/rdinmem.c @ 965:2969032bdfac

fcup-smsend[mult]: fix buglet in K&R C NULL pointer passing The only 100% safe way to pass a NULL pointer as a function argument in K&R C is to cast 0 to a pointer type; failing to do so may cause mysterious bugs (invalid stack frames or garbage in argument registers) on 64-bit machines. This issue has already been fixed in most of FC host tools, but I just found some missed spots: passing of NULL UDH to PDU encoding functions in fcup-smsend[mult] in the case of single (not concatenated) SMS.
author Mychaela Falconia <falcon@freecalypso.org>
date Fri, 01 Sep 2023 07:33:51 +0000
parents 9214118ae941
children
line wrap: on
line source

#include <sys/types.h>
#include "types.h"
#include "struct.h"
#include "globals.h"
#include "macros.h"

tiffs_read_into_ram(pathname, buf, maxlen, lenrtn)
	char *pathname;
	u8 *buf;
	size_t maxlen, *lenrtn;
{
	int stat, cont;
	u8 *chunk_start;
	size_t chunk_size, real_len, roomleft;

	stat = tiffs_find_file(pathname, &chunk_start, &chunk_size, &cont);
	if (stat < 0)
		return(stat);
	if (chunk_size > maxlen) {
toobig:		printf("Error: %s is bigger than the read buffer\n", pathname);
		return(-1);
	}
	real_len = chunk_size;
	memcpy(buf, chunk_start, chunk_size);
	buf += chunk_size;
	roomleft = maxlen - chunk_size;
	while (cont) {
		stat = tiffs_get_segment(cont, &chunk_start, &chunk_size,
					 &cont);
		if (stat < 0)
			return(stat);
		if (chunk_size > roomleft)
			goto toobig;
		real_len += chunk_size;
		memcpy(buf, chunk_start, chunk_size);
		buf += chunk_size;
		roomleft -= chunk_size;
	}
	if (lenrtn)
		*lenrtn = real_len;
	return(0);
}