fortran: Fixes a bug in ISO_Fortran_binding.c.

libgfortran/ChangeLog:

	* runtime/ISO_Fortran_binding.c (CFI_establish): Fixed signed
	  char arrays. Signed char or uint8_t arrays would cause
	  crashes unless an element size is specified.

gcc/testsuite/ChangeLog:

	* gfortran.dg/iso_fortran_binding_uint8_array.f90: New test.
	* gfortran.dg/iso_fortran_binding_uint8_array_driver.c: New test.
This commit is contained in:
Jerry DeLisle 2021-01-15 13:48:42 -08:00
parent 9beb6d88ef
commit b90e4a9741
3 changed files with 37 additions and 2 deletions

View File

@ -0,0 +1,11 @@
! { dg-do run }
! { dg-additional-sources iso_fortran_binding_uint8_array_driver.c }
module m
use iso_c_binding
contains
subroutine fsub( x ) bind(C, name="fsub")
integer(c_int8_t), intent(inout) :: x(:)
x = x+1
end subroutine
end module

View File

@ -0,0 +1,25 @@
#include <stdlib.h>
#include <stdio.h>
#include <inttypes.h>
#include "ISO_Fortran_binding.h"
extern void fsub(CFI_cdesc_t *);
int main(void)
{
int8_t x[] = {1,2,3,4};
int N = sizeof(x)/sizeof(x[0]);
CFI_CDESC_T(1) dat;
CFI_index_t ext[1];
ext[0] = (CFI_index_t)N;
int rc = CFI_establish((CFI_cdesc_t *)&dat, &x, CFI_attribute_other,
CFI_type_int8_t, 0, (CFI_rank_t)1, ext);
printf("CFI_establish call returned: %d\n", rc);
fsub((CFI_cdesc_t *)&dat );
for (int i=0; i<N; i++)
printf("%"PRId8"\n", x[i]);
return 0;
}

View File

@ -345,8 +345,7 @@ int CFI_establish (CFI_cdesc_t *dv, void *base_addr, CFI_attribute_t attribute,
dv->base_addr = base_addr;
if (type == CFI_type_char || type == CFI_type_ucs4_char ||
type == CFI_type_signed_char || type == CFI_type_struct ||
type == CFI_type_other)
type == CFI_type_struct || type == CFI_type_other)
dv->elem_len = elem_len;
else
{