Hi Tommi,

Tommi Vainikainen <[EMAIL PROTECTED]> wrote:
> In nodes.c at function node_process_handshake_header there is
> assert(rw < sizeof(gnet_response)) at least two times.

I've just run into the same problem.

> rw = g_snprintf(gnet_response, sizeof(gnet_response), ...);
> Return value of g_snprintf is number of characters (excluding the
> trailing '\0') which would have been written to the final string if
> enough space had been available.

I even assume there might be more g_sprintf()s to check whether the
return value is handled correctly.

> That means assertion failure will happen if there was not enough space
> available in fixed size buffer gnet_response, which is char[1024].

You're right, this shouldn't be necessary and handled in another way.

> For me assertion failure just happened and GTKG just killed itself.
Lucky me. I've hit 'c' into gdb after inspecting this and GTKG kept
running. :-)

Thanks for your report - should be fixed soon,
Christian

Attachment: msg00032/pgp00000.pgp
Description: PGP signature

Reply via email to