Help! free() invalid pointer error

[ahoodin]

When you don't post the whole code, you are assuming that you know where the bug is, and that precludes other people from observing all possibilities. Not posting your whole code makes the problem hard to debug by others, and your code frustrating to look at, and well sort of on you to figure out in the end.

[2kaud]

*(message*+size) = '\0';

assuming this is actual code, then should be

*(*message + size) = '\0';

but as others have requested, please post actual code.

[2kaud]

Actually, you don't really need the temp buffer nor explicity null terminate *message. Consider (not tried)

	*message = (uint8_t*)calloc(size + 1, sizeof(uint8_t));

	for (int bytes = 0, ret = 0; bytes < size; bytes += ret)
		if ((ret = recv(socket, *message + bytes, size - bytes, MSG_NOSIGNAL)) < 0)
		{
			free(*message);
			*message = NULL;
			return -1;
		}