Speicher nicht richtig verwaltet

[verwirrter Frickler (Gast)]

Nach dem Compilieren und dem Aufruf via Kommandozeile macht mein 
Programm auf einem Raspberry Pi scheinbar brav was es soll:
sudo /home/pi/libmodbus/Raspi_III_GPIO_RS-485/readSDM120 -s /dev/ttyS0 
-r30007
gibt einen Meßwert (hier: Ampere) an stdout aus:
0.1
Ich habe die zu übergebenden Parameter in vielen Kombinationen getestet 
und es gab keine Anomalien.


Wenn ich aber die gleiche Zeile via exec-Node von node-red aufrufe, 
kommt im Debug-Fenster das dabei raus:
"ERROR (ignored): Non-option argument 1504101045512↵0.1↵"
Die Fehlermeldung "ERROR (ignored): Non-option argument" habe ich zum 
Debuggen eingbaut, sie kann nur kommen wenn ein zweiteiliger Parameter 
(zB. -s /devTTYS0) nicht vollständig wäre; das ist aber hier nicht der 
Fall.

Die Konsole unter node-red gibt:
pi : TTY=unknown ; PWD=/home/pi ; USER=root ; 
COMMAND=/home/pi/libmodbus/Raspi_III_GPIO_RS-485/readSDM120 -s 
/dev/ttyS0 -r30007 1504101045512

Wo kommt diese 13-stellige Zahl her? Könnten das die Reste eines 
kaputten Zeigers o.ä. sein...?


Der Messwert kommt zwar (ganz am Schluß), aber vorher kommt Datenmüll. 
Als ob Speicher nicht freigegeben/geflusht o.ä vergessen worden wäre.

Gleich am Anfang von main werden die Kommandozeilenparameter 
ausgewertet.

Vorweg: Ich mache das Auswerten der Kommandozeilenargumente das erste 
mal, und bin darin noch ziemlich unsicher...

struct someStuff_t {
    int SDM120reg;
    int debug;
    char serial[20];
    };

int main(int argc, char *argv[])
{
  ...
  ...
  ...

    struct someStuff_t someStuff;

// Step1: Extract the selected register and serial interface
// Example:
// readSDM120 -d 1 -s /dev/ttyS0 -r 30013
// DEBUG Mode 1, svalue = /dev/ttyS0, rvalue => SDM120_POWER

    if (argc == 1) {
        printf ("Use with command line options (-h for help).\n");
        return (0);
  }

    evaluate_main (argc, argv, &someStuff);  // someStuff wird mit den ausgewerteten Kommandozeilen-Argumenten gefüllt
  ...
  ...
  ...
}





int evaluate_main (int eval_argc, char **eval_argv, struct someStuff_t *stuff)
{

  int dflag = 0;  //debug
  int sflag = 0;  //serial interface
  int rflag = 0;  //register flag
  char *svalue = NULL;  //serial interface as string
  char *rvalue = NULL;  //register value as string
  char *dvalue = NULL;  //debug mode as string : 1 or 2
  int base = 10;
  char *endptr;
  int index;
  int c;
  opterr = 0;

/* Get data from SDM120 Modbus device
 Parameter:
 -d debug mode
 -s serial
 -r Input register (3x range) start address of the slave device. Always 'address' and 'address+1' are fetched
    as IEEE-754 float is stored in 2x16 bit registers

 example: fetch current from SDM120 via /dev/ttyS0
 SDM120 -d -s /dev/ttyS0 -r 30007
*/

  while ((c = getopt (eval_argc, eval_argv, "d:hs:r:")) != -1)
    switch (c)
      {
      case 'h':
        //Help
        printf("SDM120 readout.\n");
        printf("Use with option:\n");
        printf(" -h  This help here\n");
        printf(" -d  Debug mode, 1: read single input register, 2: read all input registers\n");
        printf(" -s  Serial interface: /dev/ttyS0, /dev/ttyACM0...\n");
        printf(" -r  Register to be read: 30001, 30006, 30343...\n");
        printf("Example: SDM120 -d 1 -s /dev/ttyS0 -r 30001\n\n");
        printf("\n");
        exit ();

       case 'd':
        printf ("DEBUG mode\n");
        dvalue = optarg;
        stuff->debug = strtol(dvalue, &endptr, base);
        if (stuff->debug == 1) dflag = 1;
        break;

       case 's':
        sflag = 1;
      //  svalue = optarg;
        strcpy (stuff->serial, optarg);
        break;

      case 'r':
    //    rvalue = optarg;
        // val =
        //stuff->SDM120reg = strtol(rvalue, &endptr, base);
        stuff->SDM120reg = strtol(optarg, &endptr, base);
        rflag = 1;
        break;

      case '?':
        if ((optopt == 'r') || (optopt == 's')||(optopt == 'd'))
          fprintf (stderr, "Option -%c requires an argument.\n", optopt);
        else if (isprint (optopt))
          fprintf (stderr, "Unknown option `-%c'.\n", optopt);
        else
          fprintf (stderr,
                   "Unknown option character `\\x%x'.\n",
                   optopt);
        return 1;
      default:
        abort ();
      }

     if (stuff->debug == 1) printf ("Local: dflag = %d, sflag = %d rflag = %d, svalue = %s, rvalue = %s\n",
                dflag, sflag, rflag, svalue, rvalue);

  for (index = optind; index < eval_argc; index++)
    printf ("ERROR (ignored): Non-option argument %s\n", eval_argv[index]);

  return 0;

}

Irgendetwas geht in dieser Auswerteroutine böse daneben - hat jemand 
eine Idee?

[guest (Gast)]

Das wichtigste hast Du vergessen

verwirrter Frickler schrieb:
> Wenn ich aber die gleiche Zeile via exec-Node von node-red aufrufe

Ja wie denn? Deine node-red Konsole zeigt Dir doch, das die Zahl bereits 
als Parameter an Dein Programm übergeben wird, Deine Routine arbeitet 
also völlig korrekt und die Fehlermeldung passt auch.

[verwirrter Frickler (Gast)]

Nachdem ich im ersten Moment über deine Aussage gerätselt habe, habe ich 
doch nochmal nachgedacht - und tatsächlich: Defaultmäßig ist bei den 
Eigenschaften des Nodes "+ Append  msg.payload" angehakt. Ich habe das 
nicht gesehen.
Nun ist der Müll weg. Mein selbstgeschriebener Code funktioniert 
ausnahmsweise mal wie er soll. HA!

Vielen Dank für den Tip. Manchmal brauchts einen Anstoß...

[guest (Gast)]

Damit Du beim nächsten Mal vielleicht weniger rätseln mußt :)

"COMMAND=/home/pi/libmodbus/Raspi_III_GPIO_RS-485/readSDM120 -s
/dev/ttyS0 -r30007 1504101045512"

Das war die entscheidende Zeile in Deinem Post. Da sieht man den Aufruf 
Deines Programs samt der komischen Zahl als letzem Parameter.