ошибка Segmantation при добавлении элементов в структуре

Question

Почему я получаю ошибку сегментации в этой функции:

#include <stdio.h> 
#include <stdlib.h> 
#include "math.h" 

vec_t mtrx_multiple (sparse_mat_t a, vec_t c) { 

    vec_t result; 
    int i; 

    result.n = a.n; 
    printf("result.n: %d\n", result.n); 


    result.vec = malloc(a.n * sizeof *result.vec); 
    for(i=0; i<a.n; i++) 
     result.vec[i] = c.vec[i] * a.a[a.ja[i]]; 


    return result; 
} 

Структура:

typedef struct { 
    int n; 
    int *vec; 
} vec_t; 

typedef struct { 
    int *a; 
    int *ia; 
    int *ja; 
    int n; 
} sparse_mat_t; 

Спасибо за помощь

Answer 1

Я подозреваю, что проблема есть с a.a[a.ja[i]], вы должны попробовать проверить значения a.ja[i] перед их использованием для индекса a.a.

Было бы полезно знать, как инициализируется a, а также на какой строке возникает segfault.

Answer 2

Я подозреваю, что это линия, где беда:

 
    result.vec = malloc(a.n * sizeof *result.vec); 
    for(i=0; i<a.n; i++) 
     result.vec[i] = c.vec[i] * a.a[a.ja[i]]; 

Причина заключается в том, что вы не malloc ИНГ для каждого result.vec[i] ..

Вы можете это подтвердить?

Edit: Благодаря Alok и Devel для информирования меня о моей ошибке ... Что sizeof *result.vec возвращение? Правда, это выглядит запутанным, как если старшинство между sizeof смешивается с * ...

Надеется, что это помогает, С наилучшими пожеланиями, Том.

Answer 3

Malloc может терпеть неудачу и возвращать null. a.ja [i] может не находиться между 0 и n. Что же представляет собой массив ja, который должен представлять?

Наши спекуляции не собираются давать ответ. Запуск вашей программы под отладчиком будет.

Answer 4

 
typedef struct { 
    int n; 
    int *vec; 
} vec_t;
 

int main(int argc, char **argv) { vec_t result; int i; int size; result.n = 5; size = result.n * sizeof *result.vec; result.vec = malloc(size); for(i=0; i<result.n; i++) { result.vec[i] = i; } return i; } 

Я согласен с Autopulated, эта версия кода работает нормально, единственное, что я оставил в этом рефакторинга является а и с связанные вещи. Я бы проверял, что a и c инициализируются должным образом.