]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
Don't segfault on syscall (SYS_io_destroy, 0). (Jakub Jelinek) Fixes
authorJulian Seward <jseward@acm.org>
Sat, 17 Nov 2007 01:49:06 +0000 (01:49 +0000)
committerJulian Seward <jseward@acm.org>
Sat, 17 Nov 2007 01:49:06 +0000 (01:49 +0000)
#147325.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@7170

coregrind/m_syswrap/syswrap-linux.c

index 0d8c90a18066a8c1f9d01c0853126db130a41a61..96cbe3714935235c4320c1a73623cf04064c126d 100644 (file)
@@ -1223,17 +1223,18 @@ POST(sys_io_setup)
 // file-descriptors are closed...
 PRE(sys_io_destroy)
 {
-   struct vki_aio_ring *r;
-   SizeT size;
+   SizeT size = 0;
       
    PRINT("sys_io_destroy ( %llu )", (ULong)ARG1);
    PRE_REG_READ1(long, "io_destroy", vki_aio_context_t, ctx);
 
    // If we are going to seg fault (due to a bogus ARG1) do it as late as
    // possible...
-   r = (struct vki_aio_ring *)ARG1;
-   size = VG_PGROUNDUP(sizeof(struct vki_aio_ring) + 
-                       r->nr*sizeof(struct vki_io_event));
+   if (ML_(safe_to_deref)( (void*)ARG1, sizeof(struct vki_aio_ring))) {
+      struct vki_aio_ring *r = (struct vki_aio_ring *)ARG1;
+      size = VG_PGROUNDUP(sizeof(struct vki_aio_ring) + 
+                          r->nr*sizeof(struct vki_io_event));
+   }
 
    SET_STATUS_from_SysRes( VG_(do_syscall1)(SYSNO, ARG1) );