Unfortunately, you have run into one of the inherent limitations of CVI's run-time checking. Even though it is perfectly legal in C and somewhat common practice, our run-time checking doesn't like it when you conceptually split up a block of memory and treat it as two or more separate blocks.
While I cannot fix the problem in our run-time checking without breaking other use cases, I can explain what's causing the error and how you can work around it.
When you malloc memory, we assume that the memory block is going to hold one or more instances of the type to which you're casting the memory block. In this case, CVI is treating the new memory block as an array of arg_info structures. But your new memory block is not big enough to hold more than one instance of struct arg_info, so we implicitly truncate the memory block to sizeof(struct arg_info). Because of the implicit truncation, s->values is now pointing past the end of the memory block and any access will result in an error.
#include <ansi_c.h>
struct arg_int {
char some_large_block[64];
int count;
int *values;
};
int main (int argc, char *argv[])
{
int i, n = 4;
struct arg_int *s = malloc(sizeof *s + n * sizeof *s->values);
s->count = n;
s->values = (int *)(s+1);
for (i = 0; i < n; ++i)
s->values[i] = i;
return 0;
}
You can avoid the implicit truncation in the original code by assigning to a (void*) first. This retains the original block size by keeping the actual type of the data in the memory block in limbo. Subsequent casts do not truncate the block. We truncate only when the cast occurs implicitly as part of a malloc. s->values points to the remainder of the block and you can access it freely, but you do not get any run-time checking on it.
#include <ansi_c.h>
struct arg_int {
char some_large_block[64];
int count;
int *values;
};
int main (int argc, char *argv[])
{
int i, n = 4;
struct arg_int *s;
void *memory = malloc(sizeof *s + n * sizeof *s->values);
s = memory;
s->count = n;
s->values = (int *)(s+1);
for (i = 0; i < n; ++i)
s->values[i] = i;
return 0;
}
If you want full run-time checking on s->values, you have to allocate the two components separately.
#include <ansi_c.h>
struct arg_int {
char some_large_block[64];
int count;
int *values;
};
int main (int argc, char *argv[])
{
int i, n = 4;
struct arg_int *s = malloc(sizeof *s);
s->count = n;
s->values = malloc(n * sizeof *s->values);
for (i = 0; i < n; ++i)
s->values[i] = i;
return 0;
}
Another option is to use an incomplete array. An incomplete array is an array of unspecified or zero size at the end of a structure definition. CVI is implicitly growing the array to fill up the rest of the allocated memory block. You do not get run-time checking for the array.
#include <ansi_c.h>
struct arg_int {
char some_large_block[64];
int count;
int values[0];
};
int main (int argc, char *argv[])
{
int i, n = 4;
struct arg_int *s = malloc(sizeof *s + n * sizeof *s->values);
s->count = n;
for (i = 0; i < n; ++i)
s->values[i] = i;
return 0;
}
You can also disable run-time checking for your memory block by going through a series of casts: http://zone.ni.com/reference/en-XX/help/370051K-01/cvi/disablinguserprotectionforindividualpointer/
Best regards.
