dstLen is not set with utf8_conversions.cpp

[bxi07354@…]

src/kits/textencoding/utf8_conversions.cpp

*dstLen is not set when there are errors in convert_to_utf8() and convert_from_utf8().

It is necessary to set dstLen to 0 if there is no output.

sample if (*srcLen == 0) {

dst[0] = '\0'; *dstLen = 0; return B_OK;

}

[korli]

I tested the behavior on R5 and fixed ours in rev 16294.

[johndrinkwater]

Add (*srcLen == 0) case

[bxi07354@…]

hrev16294 is the same as R5. if (*srcLen == 0) {

*dstLen = 0; return B_ERROR;

}

hrev16328 was changed with B_OK. Why. if (*srcLen == 0) {

*dstLen = 0; return B_OK;

}

The minus was put. if (*srcLen <= 0) {

*srcLen = 0; *dstLen = 0; return B_ERROR;

}

[korli]

I think axeld checks on Dano and reproduced the behavior. Having a zero length doesn't sound as an error in the end.