On Wed, 17 May 2023 08:48:37 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
>> Xue-Lei Andrew Fan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> check returned value of snprintf > > src/utils/hsdis/binutils/hsdis-binutils.c line 246: > >> 244: >> 245: size_t used_size = snprintf(buf, bufsize, "%s", close); >> 246: if ((used_size < 0) || (used_size >= bufsize)) { > > (used_size < 0) is tautologically false, since used_size is a size_t, so > unsigned. I'm somewhat surprised > this doesn't trigger a warning from some compiler. Updated to use `int` to replace `size_t.`. Thank you for the catching. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13995#discussion_r1196646900