psxmem: fix error handling
authornotaz <notasas@gmail.com>
Tue, 24 Mar 2015 22:39:24 +0000 (00:39 +0200)
committernotaz <notasas@gmail.com>
Tue, 24 Mar 2015 23:00:53 +0000 (01:00 +0200)
also bump libpicofe for mmap fix

frontend/libpicofe
libpcsxcore/psxmem.c

index 9fec8a9..26ea181 160000 (submodule)
@@ -1 +1 @@
-Subproject commit 9fec8a91c9b19856ac0b51de53b847b38ed8dc61
+Subproject commit 26ea18173c1228dd5ce39e2a88ffe1ae10fcb365
index 86da68c..6277220 100644 (file)
@@ -49,20 +49,21 @@ void *psxMap(unsigned long addr, size_t size, int is_fixed,
 
 retry:
        if (psxMapHook != NULL) {
 
 retry:
        if (psxMapHook != NULL) {
-               ret = psxMapHook(addr, size, is_fixed, tag);
-               goto out;
+               ret = psxMapHook(addr, size, 0, tag);
+               if (ret == NULL)
+                       return NULL;
+       }
+       else {
+               /* avoid MAP_FIXED, it overrides existing mappings.. */
+               /* if (is_fixed)
+                       flags |= MAP_FIXED; */
+
+               req = (void *)addr;
+               ret = mmap(req, size, PROT_READ | PROT_WRITE, flags, -1, 0);
+               if (ret == MAP_FAILED)
+                       return NULL;
        }
 
        }
 
-       /* avoid MAP_FIXED, it overrides existing mappings.. */
-       /* if (is_fixed)
-               flags |= MAP_FIXED; */
-
-       req = (void *)addr;
-       ret = mmap(req, size, PROT_READ | PROT_WRITE, flags, -1, 0);
-       if (ret == MAP_FAILED)
-               return NULL;
-
-out:
        if (addr != 0 && ret != (void *)addr) {
                SysMessage("psxMap: warning: wanted to map @%08x, got %p\n",
                        addr, ret);
        if (addr != 0 && ret != (void *)addr) {
                SysMessage("psxMap: warning: wanted to map @%08x, got %p\n",
                        addr, ret);
@@ -72,8 +73,7 @@ out:
                        return NULL;
                }
 
                        return NULL;
                }
 
-               if (ret != NULL && ((addr ^ (long)ret) & 0x00ffffff)
-                   && !tried_to_align)
+               if (((addr ^ (long)ret) & 0x00ffffff) && !tried_to_align)
                {
                        psxUnmap(ret, size, tag);
 
                {
                        psxUnmap(ret, size, tag);