Syscall arguments are neveger negative

[randomstuff]

The arguments are defined as unsigned long:

typedef struct open_arg_s {
  unsigned long ret;
  unsigned long ptr_filename;
  unsigned long flags;
  unsigned long mode;
} open_arg_s;

However the code has a lot of tests for negativeness:

if (arg->ret >= 0) {
  // This is always executed
}

[mquinson]

In strace, the result of syscalls is a long: http://sources.debian.net/src/strace/4.10-1/defs.h/#L226

(unless I'm missing something)

[randomstuff]

Well we should either define the fields as (signed) long or cast them when checking for negativeness. Or use the correct type: for open, int might be enough.