RFR: 8314114: Fix -Wconversion warnings in os code, primarily linux [v7]

Dean Long dlong at openjdk.org
Mon Aug 14 20:38:18 UTC 2023


On Mon, 14 Aug 2023 15:04:58 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This change fixes various -Wconversion warnings from files in the os directories and related, widening types or adding checked_cast<> where narrowing.
>> Tested with tier1 on linux/macosx/windows and tier1-4 on linux-x64-debug, windows-x64-debug.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use 'long' and fix -Wsign-compare warnings from previous edit.  Make max_len size_t in the three places.

Marked as reviewed by dlong (Reviewer).

src/hotspot/os/linux/os_linux.cpp line 1608:

> 1606:     // No more space in ebuf for additional diagnostics message
> 1607:     return nullptr;
> 1608:   }

Suggestion:

  size_t ebuf_len = strlen(ebuf);
  if (ebuflen <= ebuf_len) {
    // No more space in ebuf for additional diagnostics message
    return nullptr;
  }
  size_t diag_msg_max_length = ebuflen - ebuf_len;

src/hotspot/os/linux/os_linux.cpp line 1610:

> 1608:   }
> 1609: 
> 1610:   char* diag_msg_buf=ebuf+strlen(ebuf);

Suggestion:

  char* diag_msg_buf=ebuf+ebuf_len;

-------------

PR Review: https://git.openjdk.org/jdk/pull/15229#pullrequestreview-1577611177
PR Review Comment: https://git.openjdk.org/jdk/pull/15229#discussion_r1293942588
PR Review Comment: https://git.openjdk.org/jdk/pull/15229#discussion_r1293943491


More information about the hotspot-dev mailing list