diff options
| author | Robert Morris <[email protected]> | 2019-07-28 06:29:37 -0400 |
|---|---|---|
| committer | Robert Morris <[email protected]> | 2019-07-28 06:29:37 -0400 |
| commit | 6507da772d63d4dd0e365b976f2b400d64dcfccb (patch) | |
| tree | 134216f9de126a30e58ff7dc9aea90d07d5bfba2 /kernel/sysfile.c | |
| parent | 629faafa36325bf36e153605d6bd233f3f417a69 (diff) | |
| download | xv6-labs-6507da772d63d4dd0e365b976f2b400d64dcfccb.tar.gz xv6-labs-6507da772d63d4dd0e365b976f2b400d64dcfccb.tar.bz2 xv6-labs-6507da772d63d4dd0e365b976f2b400d64dcfccb.zip | |
argptr no longer needed, since copyin checks
Diffstat (limited to 'kernel/sysfile.c')
| -rw-r--r-- | kernel/sysfile.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/kernel/sysfile.c b/kernel/sysfile.c index 2c787ec..292fa27 100644 --- a/kernel/sysfile.c +++ b/kernel/sysfile.c @@ -73,7 +73,7 @@ sys_read(void) int n; uint64 p; - if(argfd(0, 0, &f) < 0 || argint(2, &n) < 0 || argptr(1, &p, n) < 0) + if(argfd(0, 0, &f) < 0 || argint(2, &n) < 0 || argaddr(1, &p) < 0) return -1; return fileread(f, p, n); } @@ -85,7 +85,7 @@ sys_write(void) int n; uint64 p; - if(argfd(0, 0, &f) < 0 || argint(2, &n) < 0 || argptr(1, &p, n) < 0) + if(argfd(0, 0, &f) < 0 || argint(2, &n) < 0 || argaddr(1, &p) < 0) return -1; return filewrite(f, p, n); @@ -110,7 +110,7 @@ sys_fstat(void) struct file *f; uint64 st; // user pointer to struct stat - if(argfd(0, 0, &f) < 0 || argptr(1, &st, sizeof(struct stat)) < 0) + if(argfd(0, 0, &f) < 0 || argaddr(1, &st) < 0) return -1; return filestat(f, st); } @@ -453,7 +453,7 @@ sys_pipe(void) int fd0, fd1; struct proc *p = myproc(); - if(argptr(0, &fdarray, 2*sizeof(int)) < 0) + if(argaddr(0, &fdarray) < 0) return -1; if(pipealloc(&rf, &wf) < 0) return -1; |