bugID:19953636:[cm]fixed fd lock null pointer check incompleteness issue
@@ -64,7 +64,7 @@ int iotx_cm_connect(int fd, uint32_t timeout)
iotx_cm_connect_fp connect_func;
int ret;
- if (_fd_is_valid(fd) == -1) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
@@ -125,7 +125,7 @@ static int _iotx_cm_yield(int fd, unsigned int timeout)
return 0;
}
- if (_fd_is_valid(fd) == -1) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
@@ -163,7 +163,7 @@ int iotx_cm_sub(int fd, iotx_cm_ext_params_t *ext, const char *topic,
{
iotx_cm_sub_fp sub_func;
- if (_fd_is_valid(fd) == -1) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
@@ -178,7 +178,7 @@ int iotx_cm_unsub(int fd, const char *topic)
{
iotx_cm_unsub_fp unsub_func;
- if (_fd_is_valid(fd) == -1) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
@@ -195,7 +195,7 @@ int iotx_cm_pub(int fd, iotx_cm_ext_params_t *ext, const char *topic, const char
{
iotx_cm_pub_fp pub_func;
- if (_fd_is_valid(fd) == -1) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
@@ -210,7 +210,7 @@ int iotx_cm_close(int fd)
{
iotx_cm_close_fp close_func;
- if (_fd_is_valid(fd) != 0) {
+ if (_fd_is_valid(fd) < 0) {
cm_err(ERR_INVALID_PARAMS);
return -1;
}
