r.command = htons(i);
command_length = PKL_CommandLength(&r);
padding_length = PKL_CommandPaddingLength(&r);
- assert(padding_length <= MAX_PADDING_LENGTH && padding_length <= command_length);
- assert((command_length >= offsetof(CMD_Request, data) &&
- command_length <= sizeof (CMD_Request)) || command_length == 0);
+ if (padding_length > MAX_PADDING_LENGTH || padding_length > command_length ||
+ command_length > sizeof (CMD_Request) ||
+ (command_length && command_length < offsetof(CMD_Request, data)))
+ assert(0);
}
for (i = 1; i < N_REPLY_TYPES; i++) {
r.status = STT_SUCCESS;
r.data.manual_list.n_samples = htonl(MAX_MANUAL_LIST_SAMPLES);
reply_length = PKL_ReplyLength(&r);
- assert((reply_length >= offsetof(CMD_Reply, data) &&
- reply_length <= sizeof (CMD_Reply)) || reply_length == 0);
+ if ((reply_length && reply_length < offsetof(CMD_Reply, data)) ||
+ reply_length > sizeof (CMD_Reply))
+ assert(0);
}
sock_fdu = -1;
/* Don't want to allow the same fd to register a handler more than
once without deleting a previous association - this suggests
a bug somewhere else in the program. */
- assert(!FD_ISSET(fd, &read_fds));
+ if (FD_ISSET(fd, &read_fds))
+ assert(0);
++n_read_fds;
assert(initialised);
/* Check that a handler was registered for the fd in question */
- assert(FD_ISSET(fd, &read_fds));
+ if (!FD_ISSET(fd, &read_fds))
+ assert(0);
--n_read_fds;