Project

General

Profile

Actions

Bug #3124

closed

Analyse Logiscope LFR_3.2.0.21 : LFR_basic-parameters : Don_Initialisation_P1 Severity is High

Added by William Recart about 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
03/10/2018
Due date:
% Done:

0%

Estimated time:
revision:
r0

Description

Rappel de la règle :
Don_Initialisation_P1:
Definition:
-----------
Global variables must be initialized when they are defined.

Justification:
--------------
Not all compilers give the same default values. Unexpected behaviour can be
avoided with better control over variable values. Initializing global variables
when they are declared ensures that they are initialized before being used.

La règle n'est pas respectée dans 6 cas d'après Logiscope:
basic_parameters_utilities.h : lines : 19, 20, 21, 22, 24, 25

Actions #1

Updated by William Recart about 6 years ago

Exemple de correction :
float compressed_spectral_matrix_f0[NB_BINS_COMPRESSED_MATRIX_f0 * NB_VALUES_PER_SPECTRAL_MATRIX] = {0.0} ;

Actions #2

Updated by bruno katra almost 6 years ago

  • Status changed from New to In Progress

Corrigé, en attente de validation par Thomas.

Actions #3

Updated by Alexis Jeandet almost 6 years ago

Ce source n'est pas utilisé dans le FSW, c'est juste utilisé pour les tests de Thomas un local sur son PC.

Les seuls sources à analyser et corriger sont:

- LFR_basic-parameters/basic_parameters.c
- LFR_basic-parameters/basic_parameters.h
- LFR_basic-parameters/basic_parameters_params.h

De plus le problème n'est pas tant l'init que la déclaration de variables dans un header...

Actions #4

Updated by bruno katra almost 6 years ago

  • Status changed from In Progress to Closed

Alexis Jeandet wrote:

Ce source n'est pas utilisé dans le FSW, c'est juste utilisé pour les tests de Thomas un local sur son PC.

Les seuls sources à analyser et corriger sont:

- LFR_basic-parameters/basic_parameters.c
- LFR_basic-parameters/basic_parameters.h
- LFR_basic-parameters/basic_parameters_params.h

De plus le problème n'est pas tant l'init que la déclaration de variables dans un header...

En effet ! J'avais pas fait attention que c'était un .h et en plus que ce fichier sert uniquement à Thomas en local.
On peut donc cloturer ce point puisque ce code ne fait pas partie du FSW...

Actions

Also available in: Atom PDF