mirror of
https://github.com/bytecodealliance/wasm-micro-runtime.git
synced 2025-09-05 17:32:26 +00:00
fix: cast file descriptor to size_t for comparison in fd_prestats functions (#4582)
``` comparison of integer expressions of different signedness: ‘__wasi_fd_t’ {aka ‘int’} and ‘size_t’ {aka ‘long unsigned int’} [-Werror=sign-compare] 288 | if (fd >= pt->size) ```
This commit is contained in:
parent
0b97d0cb14
commit
9cb1cc4af6
|
@ -285,7 +285,7 @@ fd_prestats_get_entry(struct fd_prestats *pt, __wasi_fd_t fd,
|
||||||
struct fd_prestat **ret) REQUIRES_SHARED(pt->lock)
|
struct fd_prestat **ret) REQUIRES_SHARED(pt->lock)
|
||||||
{
|
{
|
||||||
// Test for file descriptor existence.
|
// Test for file descriptor existence.
|
||||||
if (fd >= pt->size)
|
if ((size_t)fd >= pt->size)
|
||||||
return __WASI_EBADF;
|
return __WASI_EBADF;
|
||||||
struct fd_prestat *prestat = &pt->prestats[fd];
|
struct fd_prestat *prestat = &pt->prestats[fd];
|
||||||
if (prestat->dir == NULL)
|
if (prestat->dir == NULL)
|
||||||
|
@ -301,7 +301,7 @@ static __wasi_errno_t
|
||||||
fd_prestats_remove_entry(struct fd_prestats *pt, __wasi_fd_t fd)
|
fd_prestats_remove_entry(struct fd_prestats *pt, __wasi_fd_t fd)
|
||||||
{
|
{
|
||||||
// Test for file descriptor existence.
|
// Test for file descriptor existence.
|
||||||
if (fd >= pt->size)
|
if ((size_t)fd >= pt->size)
|
||||||
return __WASI_EBADF;
|
return __WASI_EBADF;
|
||||||
struct fd_prestat *prestat = &pt->prestats[fd];
|
struct fd_prestat *prestat = &pt->prestats[fd];
|
||||||
|
|
||||||
|
@ -356,7 +356,7 @@ fd_table_get_entry(struct fd_table *ft, __wasi_fd_t fd,
|
||||||
REQUIRES_SHARED(ft->lock)
|
REQUIRES_SHARED(ft->lock)
|
||||||
{
|
{
|
||||||
// Test for file descriptor existence.
|
// Test for file descriptor existence.
|
||||||
if (fd >= ft->size) {
|
if ((size_t)fd >= ft->size) {
|
||||||
return __WASI_EBADF;
|
return __WASI_EBADF;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user