flush

[m6w6/ext-psi] / src / libffi.c

diff --git a/src/libffi.c b/src/libffi.c
index 44c591bfdade94dcfb23a420413803da157308c5..301995e5903d9bc377c3d2630f1be48fed9e0513 100644 (file)
--- a/src/libffi.c
+++ b/src/libffi.c
@@ -8,6 +8,7 @@
 
 #include "php_psi.h"
 #include "libffi.h"
+#include "engine.h"
 
 #undef PACKAGE
 #undef PACKAGE_BUGREPORT
@@ -87,6 +88,7 @@ static inline ffi_type *psi_ffi_token_type(token_t t) {
        case PSI_T_BOOL:
                return &ffi_type_uchar;
        case PSI_T_INT:
+       case PSI_T_ENUM:
                return &ffi_type_sint;
        case PSI_T_LONG:
                return &ffi_type_slong;
@@ -94,6 +96,10 @@ static inline ffi_type *psi_ffi_token_type(token_t t) {
                return &ffi_type_float;
        case PSI_T_DOUBLE:
                return &ffi_type_double;
+#ifdef HAVE_LONG_DOUBLE
+       case PSI_T_LONG_DOUBLE:
+               return &ffi_type_longdouble;
+#endif
        case PSI_T_POINTER:
                return &ffi_type_pointer;
        }
@@ -127,53 +133,58 @@ static void psi_ffi_struct_type_dtor(void *type) {
        free(strct);
 }
 
+static size_t psi_ffi_struct_type_pad(ffi_type **els, size_t padding) {
+       size_t i;
+
+       for (i = 0; i < padding; ++i) {
+               ffi_type *pad = malloc(sizeof(*pad));
+
+               memcpy(pad, &ffi_type_schar, sizeof(*pad));
+               *els++ = pad;
+       }
+
+       return padding;
+}
+
 static ffi_type **psi_ffi_struct_type_elements(decl_struct *strct) {
-       size_t i, j, argc = strct->args->count << 2, nels = 0, offset = 0, align, padding;
+       size_t i, argc = strct->args->count, nels = 0, offset = 0, maxalign = 0;
        ffi_type **els = calloc(argc + 1, sizeof(*els));
 
        for (i = 0; i < strct->args->count; ++i) {
                decl_arg *darg = strct->args->args[i];
                ffi_type *type = malloc(sizeof(*type));
+               size_t padding;
 
                memcpy(type, psi_ffi_decl_arg_type(darg), sizeof(*type));
 
-               if (darg->layout->pos > offset) {
-                       padding = darg->layout->pos - offset;
-                       align = ((padding - 1) | (type->alignment - 1)) + 1;
-                       if (align >= padding) {
-                               padding = 0;
-                       }
-               } else {
-                       align = 0;
-                       padding = 0;
-               }
+               ZEND_ASSERT(type->size == darg->layout->len);
 
-               if (padding) {
-                       for (j = 0; j < padding; ++j) {
-                               ffi_type *pad = malloc(sizeof(*pad));
+               if (type->alignment > maxalign) {
+                       maxalign = type->alignment;
+               }
 
-                               ZEND_ASSERT(nels + 1 < argc);
-                               memcpy(pad, &ffi_type_schar, sizeof(*pad));
-                               els[nels++] = pad;
+               if ((padding = psi_offset_padding(darg->layout->pos - offset, type->alignment))) {
+                       if (nels + padding + 1 > argc) {
+                               argc += padding;
+                               els = realloc(els, (argc + 1) * sizeof(*els));
+                               els[argc] = NULL;
                        }
+                       psi_ffi_struct_type_pad(&els[nels], padding);
+                       nels += padding;
+                       offset += padding;
                }
+               ZEND_ASSERT(offset == darg->layout->pos);
 
-               ZEND_ASSERT(nels + 1 < argc);
+               offset = (offset + darg->layout->len + type->alignment - 1) & ~(type->alignment - 1);
                els[nels++] = type;
-
-               offset += MAX(align, padding) + darg->layout->len;
        }
 
+       /* apply struct alignment padding */
+       offset = (offset + maxalign - 1) & ~(maxalign - 1);
+
        ZEND_ASSERT(offset <= strct->size);
        if (offset < strct->size) {
-               padding = strct->size - offset;
-               for (j = 0; j < padding; ++j) {
-                       ffi_type *pad = malloc(sizeof(*pad));
-
-                       ZEND_ASSERT(nels + 1 < argc);
-                       memcpy(pad, &ffi_type_schar, sizeof(*pad));
-                       els[nels++] = pad;
-               }
+               psi_ffi_struct_type_pad(&els[nels], strct->size - offset);
        }
 
        return els;
@@ -181,12 +192,13 @@ static ffi_type **psi_ffi_struct_type_elements(decl_struct *strct) {
 static inline ffi_type *psi_ffi_decl_type(decl_type *type) {
        decl_type *real = real_decl_type(type);
 
-       if (real->type == PSI_T_STRUCT) {
+       switch (real->type) {
+       case PSI_T_STRUCT:
                if (!real->strct->engine.type) {
                        ffi_type *strct = calloc(1, sizeof(ffi_type));
 
                        strct->type = FFI_TYPE_STRUCT;
-                       strct->size = real->strct->size;
+                       strct->size = 0;
                        strct->elements = psi_ffi_struct_type_elements(real->strct);
 
                        real->strct->engine.type = strct;
@@ -194,8 +206,13 @@ static inline ffi_type *psi_ffi_decl_type(decl_type *type) {
                }
 
                return real->strct->engine.type;
+
+       case PSI_T_UNION:
+               return psi_ffi_decl_arg_type(real->unn->args->args[0]);
+
+       default:
+               return psi_ffi_token_type(real->type);
        }
-       return psi_ffi_token_type(real->type);
 }
 static inline ffi_type *psi_ffi_decl_arg_type(decl_arg *darg) {
        if (darg->var->pointer_level) {