[C#] Segfault popolando un array

[Saturnix]

Ciao a tutti,

avrei bisogno di una mano con il seguente script. Come potete vedere, attraverso l'uso di atoi(argv[1, 2, 3, 4, ecc...]) estraggo vari long long int da linea di comando. Assumendo che il secondo sarà sempre maggiore del primo, creo un array delle dimensioni specificate dall'intervallo tra questi due numeri.

if (argc != 5) {
        printf("ERROR: You must provide 3 integers and a txt!\n");
        return 1;
    }
    
    long long int z = atoi(argv[1]);
    long long int b = atoi(argv[2]);
    long long int c = atoi(argv[3]);
    long long int sum = 0;
    long long int i, p, m;
    
    
    long long int tmp1 = z;
    long long int tmp2 = b;
    long long int tmp3 = c;
    
    char *userFile = argv[4];
    
    // B > Z
    long long int numbers[b-z]; 

Il problema giunge ora, quando cerco di popolare l'array con tutti i valori e mi becco il seg fault:

    for (i=0; tmp1<=tmp2; i++, tmp1++) {
        
        numbers[i] = tmp1;
    } 

il resto dello script mi è costato notti insonni e mi è impossibile usarlo a causa di questo stupidissimo bug che non riesco a scovare. Aiuto!

Grazie,
J.

EDIT:

mi son dimenticato: se fornisco allo script un input semplice funziona ("/a.out 0 12 123 a" ad esempio va) ma appena uso un numero grande non va più: ("/a.out 12 2345672 123 a" è dove mi da il problema).

[alemoppo]

Premetto che il c# non l'ho mai usato (e non lo conosco minimamente).

Comunque, prova a sostituire "<=" con "<" nel for.
Un esempio:

Mettiamo che tmp2=2 e tmp1=1. L'array verrà dimensionato di una cella (numbers[1]), mentre il ciclo viene eseguito 2 volte: 1<=2 e 2<=2, andando a scrivere in una cella non allocata.

Però è strano, perché questo errore lo dovrebbe fare anche con numero piccoli. Quindi, prova (per sicurezza ^^) a stampare z e b, per vedere se l'atoi() converte bene i numeri molto grandi.

@binary: infatti anche a me sembra C!.. è identico (a parte il fatto di dichiarare le variabili dopo il codice, che non lo fa sembrare ANSI C...

Ciao!

[binarysun]

Più che C# mi sembra C

atoi converte le stringhe in INTEGER e quindi la stringa non può essere più grande di 32767
Probabilmente ti va in errore nel cast.

[Saturnix]

Più che C# mi sembra C

atoi converte le stringhe in INTEGER e quindi la stringa non può essere più grande di 32767
Probabilmente ti va in errore nel cast.

Grande, non lo sapevo! Però nel resto dello script il numero arriva intatto ugualmente, indipendentemente dalla sua dimensione (lo verifico inserendo "printf" ogni tanto).

[dreadnaut]

Se vuoi i numeri fra B e Z inclusi gli estremi devi dichiarare

    long long int numbers[b-z+1]; 

e.g. da 5 a 10 ci sono sei numeri: 5 6 7 8 9 10