Pointerübergabe an Funktion / Sporadisch fehlerhafte Funktion

Pascal Schröer Gast #3660147
Lesenswert?
• ▲
▼ 
Hi @ all,
ich habe eine Funktion geschrieben, die fehlerhafte Parameterübergaben 
an Funktionen verhindern soll. Normalerweise läuft diese Funktion auch… 
Aber wenn ich sie z.B. 2 Mal hintereinander aufrufe, kommt es zu einer 
Fehlfunktion. Hier die Funktion:
1
void value_error(int32_t *value_in,int32_t value_min,int32_t value_max,uint32_t error_code)

2
{

3
  if(*value_in < value_min)

4
  {

5
    *value_in = value_min;

6
    error_struct.general_error_struct_1.VALUE_ERROR &= ~(error_code);  

7
  }

8
  if(*value_in > value_max)

9
  {

10
    *value_in = value_max;

11
    error_struct.general_error_struct_1.VALUE_ERROR &= ~(error_code);  

12
  }

13
}

Kann mir dazu jemand helfen? Die Fehlfunkion äußert sich darin, dass der 
Wert von *value_in unmögliche Werte annimmt.

Hier der Funktionsaufruf:
1
void PIT_init(uint8_t module, uint8_t status, uint16_t time_ms, uint8_t interrupt_en)

2
{

3
value_error((int32_t*)&module,0,3,VALUE_ERROR_PIT_init); //Funktioniert

4
value_error((int32_t*)&status,0,1,VALUE_ERROR_PIT_init); //Fehlfunktion

5
…..

Dankeschön! :)
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
Easylife Gast #3660346
Lesenswert?
• ▲
▼ 
1
int32_t value_error(int32_t value_in, int32_t value_min, int32_t 

2
value_max, uint32_t error_code)

3
{

4
  int32_t ret = value_in;

5
  if(value_in < value_min)

6
  {

7
    ret = value_min;

8
    error_struct.general_error_struct_1.VALUE_ERROR &= ~(error_code);

9
  } 

10
  else if(value_in > value_max)

11
  {

12
    value_in = value_max;

13
    error_struct.general_error_struct_1.VALUE_ERROR &= ~(error_code);

14
  }

15


16
  return (ret);

17
}

18


19
(...)

20


21
module = value_error(module, 0, 3, VALUE_ERROR_PIT_init);
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #3660445
Lesenswert?
• ▲
▼ 
Peter II schrieb:

> hier wird der 8bit wert der nur auf 32bit gecastet ist, mit 32bit
> verglichen. Keiner kann sagen was in den restlichen 24bit steht.

Doch. Kann man.

Da hier kein Pointer gecastet wird, sondern der Wert selber, wird er 
kontrolliert und entsprechend den C Regeln auf 32 Bit aufgeblasen.

Bei Casts muss man unterscheiden zwischen Casts, die nur den Datentyp 
verändern und solchen, die auch eine tatsächliche Aktion, wie eine 
Wertveränderung, nach sich ziehen.
Einen Pointer umcasten gehört zur ersten Kategorie. Einen Wert umcasten 
gehört normalerweise zur zweiten Kategorie.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 12 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.