Größe einer char wird überschritten?

[Peter H. (borntopizza)]

Moin Moin,

Ich stehe hier vor einem kleinen Problem.

Ich habe hier 2 Atmegas 328 die über 2 Enc28j60 mit dem Netzwerk 
verbunden sind.
Der eine Misst die Temperatur und gibt dann, wenn ein bestimmter 
Sollwert erreicht ist auf seiner Website "Status1" bzw. "Status0" aus, 
der 2. Atmega schaltet dann ein Relais. Das ganze funktioniert so weit 
auch ganz gut. Nur möchte ich noch 2 weitere Buttons einfügen, die GPIO 
Pins am Server High bzw. Low schalten.
Das überschreitet aber anscheinend die Größe der const char "reply" des 
Clients.
Bis 3 Buttons geht es gerade so noch, darüber werden die Daten zwar 
empfangen, aber das Relais wird nicht mehr geschalten.

Ich habe versucht den Text auch schon maximal zu kürzen, für einen 4. 
Button reicht es nicht :-(

Gibt es eine möglichkeit die Buttons oder die "Homepage" noch weiter zu 
kürzen, damit der Inhalt der char nicht "überschritten" wird?

Danke und Viele Grüße

Peter

static word homePage() {
  bfill = ether.tcpOffset();
  bfill.emit_p(PSTR(
    "HTTP/1.0 200 OK\r\n"
    "Content-Type: text/html\r\n"
    "Pragma: no-cache\r\n"
    "\r\n"
    "<meta http-equiv='refresh' content='1'/>"
    "<title>SH</title>"
    "<h1>Ti $D</h1>"
    "<h1>Ts $D</h1>"
    "<h1>Status$D</h1>"
    "<input type=submit value='+' style=width:90px;height:45px onClick=location.href'/?1'>"
    "<input type=submit value='-' style=width:90px;height:45px onClick=location.href'/?2'>"
    "<input type=submit value='R' style=width:90px;height:45px onClick=location.href'/?3'>"
    ),
      t, Tsoll, Status);
  return bfill.position();
}

[Stefan F. (Gast)]

Das sind ungefähr 420 Zeichen zuzüglich die beiden Werte für $D.

Der Mikrocontroller hat insgesamt 2kB RAM. Da kann es durchaus sein, 
dass die Grösse deines Pufferspeichers schon überschritten hast. In 
diesem Fall wirst du deine Sendung wohl in mehrere Pakete aufteilen 
müssen, das ist ja auch der Haupt-Zweck von TCP (verglichen mit UDP).

Wie gross ist denn der Puffer?

[Peter H. (borntopizza)]

Danke für die Antwort, meinst Du das hier:

byte Ethernet::buffer[700];

[Stefan F. (Gast)]

Peter H. schrieb:
> Danke für die Antwort, meinst Du das hier:
> byte Ethernet::buffer[700];

Sieht danach aus. Genaueres kann ich Dir dazu auch nicht sagen, weil ich 
deine Software nicht kenne.

Bedenke, dass in die 700 Bytes auch noch der Ethernet Header rein muss.

[Peter H. (borntopizza)]

Hmm ok.

Ich habe auch mal versucht die Zahl nach oben zu setzen, auf 1000, 
leider noch ohne Erfolg.

Das komische ist ja eigentlich, dass der Teil "Status1" bzw. "Stauts0" 
immernoch beim Client ankommt. Das kann ich im Seriellen Monitor sehen. 
Nur ab einem bestimmten Teil wird der Rest abgeschnitten und die "Reply" 
irgendiwie nicht mehr durchsucht und somit auch kein "Stauts1" bzw. 
"Status0" gefunden.

Der Server:

#include <EtherCard.h>
#include "DHT.h"
#include <EEPROM.h>

#define DHTPIN 4
#define DHTTYPE DHT11
DHT dht(DHTPIN, DHTTYPE);
unsigned long previousMillis = 0;
int Tsoll;
int t;
float Tmp;
byte value;
int Status;
static byte mymac[] = { 0x74,0x69,0x69,0x2D,0x38,0x31 };
static byte myip[] = { 192,168,10,199 };


byte Ethernet::buffer[1000];
BufferFiller bfill;

void setup (){ 
  // Change 'SS' to your Slave Select pin, if you arn't using the default pin
  if (ether.begin(sizeof Ethernet::buffer, mymac, 8) == 0)
    Serial.println(F("Failed to access Ethernet controller"));
  ether.staticSetup(myip);
  dht.begin();
  pinMode(5,OUTPUT);
  pinMode(6,OUTPUT);
  pinMode(7,OUTPUT);
  value = EEPROM.read(1);
  Tsoll = value;
  
}

static word homePage() {
  bfill = ether.tcpOffset();
  bfill.emit_p(PSTR(
    "HTTP/1.0 200 OK\r\n"
    "Content-Type: text/html\r\n"
    "Pragma: no-cache\r\n"
    "\r\n"
    "<meta http-equiv='refresh' content='1'/>"
    "<title>SH</title>"
    "<h1>Ti $D</h1>"
    "<h1>Ts $D</h1>"
    "<h1>Status$D</h1>"
    "<input type=submit value='+' style=width:90px;height:45px onClick=location.href'/?1'>"
    "<input type=submit value='-' style=width:90px;height:45px onClick=location.href'/?2'>"
    "<input type=submit value='R' style=width:90px;height:45px onClick=location.href'/?3'>"
    "<input type=submit value='H' style=width:90px;height:45px onClick=location.href'/?5'>"
    ),
      t, Tsoll, Status);
  return bfill.position();
}

void loop () {
    word len = ether.packetReceive();
    word pos = ether.packetLoop(len);
    unsigned long currentMillis = millis();
    if (currentMillis - previousMillis >= 3000) {
    previousMillis = currentMillis;
    Tmp = dht.readTemperature();
    value = EEPROM.read(1);
    }
    t = Tmp;

    if (Tsoll-2>=t){Status=1;}
    if (Tsoll-2<=t){Status=0;} 


    // IF LED10=ON turn it ON
    if(strstr((char *)Ethernet::buffer + pos, "GET /?1") != 0) {
    Tsoll++;
    EEPROM.write(1, Tsoll);
    };

    // IF LED10=OFF turn it OFF
    if(strstr((char *)Ethernet::buffer + pos, "GET /?2") != 0) {
    Tsoll--;
    EEPROM.write(1, Tsoll);
    };
    
    // IF LED10=ON turn it ON
    if(strstr((char *)Ethernet::buffer + pos, "GET /?3") != 0) {
    digitalWrite(6, LOW);
    digitalWrite(7, LOW);
    digitalWrite(5, HIGH);
    };
    // IF LED10=ON turn it ON
    if(strstr((char *)Ethernet::buffer + pos, "GET /?4") != 0) {
    digitalWrite(5, LOW);
    digitalWrite(7, LOW);
    digitalWrite(6, HIGH);
    };
    // IF LED10=ON turn it ON
    if(strstr((char *)Ethernet::buffer + pos, "GET /?5") != 0) {
    digitalWrite(5, LOW);
    digitalWrite(6, LOW);
    digitalWrite(7, HIGH);
    };

    if (pos)  // check if valid tcp data is received
    ether.httpServerReply(homePage()); // send web page data
}

Der Client:

// IoT by using module in client mode to send and receive data to your website.

// Simple demo for feeding some random data to Pachube.
// 2011-07-08 <[e-mail]> http://opensource.org/licenses/mit-license.php

// Handle returning code and reset ethernet module if needed
// 2013-10-22 [e-mail]

// Modifing so that it works on my setup for www.thingspeak.com.
// Arduino pro-mini 5V/16MHz, ETH modul on SPI with CS on pin 10.
// Also added a few changes found on various forums. Do not know what the 
// res variable is for, tweaked it so it works faster for my application
// 2015-11-09 [e-mail]

#include <EtherCard.h>

// change these settings to match your own setup
//#define FEED "000"
#define APIKEY "beef1337beef1337" // put your key here
#define ethCSpin 8 // put your CS/SS pin here.

// ethernet interface mac address, must be unique on the LAN
static byte mymac[] = { 0x74,0x69,0x69,0x2D,0x30,0x37 };
const char website[] PROGMEM = "192.168.10.199";  //Change to your domain name 
byte Ethernet::buffer[1000];
uint32_t timer;
Stash stash;
byte session;
String d1;
//timing variable
int res = 100; // was 0
int analog=0;
unsigned long previousMillis = 0;

void(* resetFunc) (void) = 0;

void setup () {
  pinMode(3,INPUT_PULLUP);
  pinMode(7,OUTPUT);
  pinMode(4,OUTPUT);
  Serial.begin(9600);
  Serial.println("\n[ThingSpeak example]");

  //Initialize Ethernet
  initialize_ethernet();

    #if 1

  // if website is a string containing an IP address instead of a domain name,
  // then use it directly. Note: the string can not be in PROGMEM.
  char websiteIP[] = "http://192.168.10.199/";
  ether.parseIp(ether.hisip, websiteIP);
#else
  // or provide a numeric IP address instead of a string
  byte hisip[] = {192.168.10.199};
  ether.copyIp(ether.hisip, hisip);
#endif

  ether.printIp("SRV: ", ether.hisip);


}


void loop () { 
  //if correct answer is not received then re-initialize ethernet module
  if (res > 220){
    initialize_ethernet(); 
  }
  
  res = res + 1;
  
  ether.packetLoop(ether.packetReceive());
  
  //200 res = 30 seconds (150ms each res)
  if (res == 200) {

    analog = analogRead(A0);
    if(digitalRead(3)) {d1 = "OFF";} else {d1 = "ON";}  

    // generate 3 values as payload - by using a separate stash,
    // we can determine the size of the generated message ahead of time
    
    byte sd = stash.create();

    stash.print("field1=");
    stash.print(highByte(analog));
    stash.print("&field2=");
    stash.print(lowByte(analog));
    stash.print("&field3=");
    stash.print(d1);
    
    stash.save();

    // generate the header with payload - note that the stash size is used,
    // and that a "stash descriptor" is passed in as argument using "$H"
    Stash::prepare(PSTR("POST /a21.php HTTP/1.1" "\r\n"
      "Host: $F" "\r\n"
      "Connection: close" "\r\n"
      "X-THINGSPEAKAPIKEY: $F" "\r\n"
      "Content-Type: application/x-www-form-urlencoded" "\r\n"
      "Content-Length: $D" "\r\n"
      "\r\n"
      "$H"),
      website, PSTR(APIKEY), stash.size(), sd);

    // send the packet - this also releases all stash buffers once done
    session = ether.tcpSend(); 

  }
  
   const char* reply = ether.tcpReply(session);
   
   if (reply != 0) {
     res = 0;
     Serial.println(F(" >>>REPLY recieved...."));
     Serial.println(reply);
      if(find(reply,"Status1")){digitalWrite(7, HIGH);digitalWrite(4, HIGH);delay(300);Serial.println("4+7HIGH");}
      if(find(reply,"Status0")){digitalWrite(7, LOW);digitalWrite(4, LOW);delay(300);}
      previousMillis = millis();
   }
   else {
      if (millis() - previousMillis > 50000) {
      previousMillis = millis();
      digitalWrite(7, LOW);digitalWrite(4, LOW);
      initialize_ethernet();
      resetFunc();
      }
   }
   delay(150);
}

boolean find(String string, String value){
  if(string.indexOf(value)>=0)
    return true;
  return false;
}

void initialize_ethernet(void){  
  for(;;){ // keep trying until you succeed 

    if (ether.begin(sizeof Ethernet::buffer, mymac, ethCSpin) == 0){ 
      Serial.println( "Failed to access Ethernet controller");
      digitalWrite(7, LOW);digitalWrite(4, LOW);
      
      continue;
    }
    
    if (!ether.dhcpSetup()){
      Serial.println("DHCP failed");
      digitalWrite(7, LOW);digitalWrite(4, LOW);
      continue;
    }

    ether.printIp("IP:  ", ether.myip);
    ether.printIp("GW:  ", ether.gwip);  
    ether.printIp("DNS: ", ether.dnsip);  

    if (!ether.dnsLookup(website))
      Serial.println("DNS failed");
      digitalWrite(7, LOW);digitalWrite(4, LOW);

    ether.printIp("SRV: ", ether.hisip);
    

    //reset init value
    res = 180;
    break;
  }
}

[PittyJ (Gast)]

Manchmal muss man eben eine Nummer größer benutzen.
Gerade im Low-End Arm Bereich gibt es Prozessoren mit mehr Ram, die 
nicht viel teurer sind.
ST-Development Boards gibt es bereits ab 10 Euro.
Ein ESP32 hat 512 KByte Ram, Wlan und kostet auch nur 10 Euro. Das 
bringt vielleicht mehr, als eine verkrüppelte Webseite, weil man nur 
2KByte Ram auf dem Atmel hat.

[Peter H. (borntopizza)]

Liegt das wirklich am Ram?

Ich konnte den Buffer bis auf 1300 erhöhen, aber das Relais mag immer 
noch nicht anziehen. Die Antwort kommt "Status1" oder "Status0" kommt ja 
auch an.

[Stefan F. (Gast)]

Da muss sicher mehr angepasst werden, als diese eine Zahl. Ausserdem 
bekommst du einen Stack-Überlauf (Absturz) wenn du den Puffer zu gross 
machst.

Selbst wenn du den Puffer jetzt grösser machst, wirst du bald wieder an 
seine Grenzen stossen. Mehr als 1,4kB ist über gewöhnliche Internet 
Anschlüsse sowieso nicht möglich.

Finde einen Weg, mit dem Puffer aufzukommen - also deine HTML Dokumente 
in mehreren Stücken zu senden.

[Paul (Gast)]

Moin, moin,

die 700 Byte waren ja gar nicht mal so schlecht.

Ich würde mir als erstes mit Wireshark ansehen, was hin und her gesendet 
wird.

Viele Grüße
Paul

[Hau Wech (Gast)]

Mehrere Pakete :

if ( bfill.emit_p(PSTR( "Etwas Text" )) ) {
      bfill.emit_p(PSTR( "Mehr Text" )) ) ;
} else {
bfill.emit_p(PSTR( "Fehlercode 1" )) ) ;
return -1;
}


Ich bin leider nur "Hacker" , kein Programmierer ..

[Peter H. (borntopizza)]

Hau Wech schrieb:
> Mehrere Pakete :
>
> if ( bfill.emit_p(PSTR( "Etwas Text" )) ) {
>       bfill.emit_p(PSTR( "Mehr Text" )) ) ;
> } else {
> bfill.emit_p(PSTR( "Fehlercode 1" )) ) ;
> return -1;
> }
>
>
> Ich bin leider nur "Hacker" , kein Programmierer ..


Hmm irgendwie mag er nicht...

Egal wie ichs drehe, ich bekomms nicht hin, immer wieder andere 
Fehlermeldungen...

[Stefan F. (Gast)]

Solange du die Fehlermeldungen nicht zitierst und nicht deinen gesamten 
Quelltext zeigst, können wir Dir kaum helfen. Dann müssen wir genau so 
blöd herum raten, wie du.

[Peter H. (borntopizza)]

static word homePage() {
    bfill = ether.tcpOffset();
 if (bfill.emit_p(PSTR(
    "HTTP/1.0 200 OK\r\n"
    "Content-Type: text/html\r\n"
    "Pragma: no-cache\r\n"
    "\r\n"
    "<meta http-equiv='refresh' content='1'/>"
    "<title>SH</title>"
    "<h1>Ti $D</h1>"
    "<h1>Ts $D</h1>"
    "<h1>Status$D</h1>"
    "<input type=submit value='+' style=width:90px;height:45px onClick=location.href'/?1'>"
    "<input type=submit value='-' style=width:90px;height:45px onClick=location.href'/?2'>"
    "<input type=submit value='S' style=width:90px;height:45px onClick=location.href'/?4'>"
    ),
    t, Tsoll, Status)
    )
  {bfill.emit_p(PSTR("<input type=submit value='S' style=width:90px;height:45px onClick=location.href'/?5'>"));} 
  else {bfill.emit_p(PSTR("Fehlercode 1"));
  return -1;}    
  return bfill.position();
}

Fehlermeldung:
could not convert 'bfill.BufferFiller::emit_p(({...}), t, Tsoll, 
Status)' from 'void' to 'bool'
Zeile ist: "t, Tsoll, Status)"

Das komische ist ja, dass ich die Website viel größer machen kann. Am PC 
wird das auch so dargestellt, nur der 2. Atmega hat Probleme das 
auszuwerten oder so.

[Einer K. (Gast)]

Wenn du die Definition von emit_p() suchst, wirst du feststellen, dass 
dessen Rückgabewert void ist.
Dein Programm erwartet allerdings bool.

Der fatale Fehler ist damit berechtigt.

[Stefan F. (Gast)]

Du fragst mit if() das Ergebnis von bfill.emit_p() ab, aber diese 
Funktion liefert gar kein Ergebnis!

Bist du sicher, dass du sie einfach so mehrmals hintereinander aufrufen 
kannst, ohne zu warten, bis der buffer gesendet wurde?

[Frank M. (ukw) (Moderator)]

Peter H. schrieb:
> could not convert 'bfill.BufferFiller::emit_p(({...}), t, Tsoll,
> Status)' from 'void' to 'bool'

Offenbar ist die Funktion emit_p() vom Typ void, gibt also keinen Wert 
zurück. Du fragst aber mit "if" den Rückgabewert der Funktion ab.

Was hat das jetzt mit der Größe des RAMs zu tun? Richtig, gar nichts.

[Stefan F. (Gast)]

>  Am PC wird das auch so dargestellt, nur der 2. Atmega hat
> Probleme das auszuwerten oder so.

Was wertet der 2. Atmega denn aus? Noch hast du uns gar nicht erklärt, 
wie deine Schaltung aussieht und welche Kommunikationsbeziehungen 
bestehen. Zudem fehlen immer noch Schaltplan und Quelltext.

Wer Hilfe erbittet, muss die dazu nötigen Infos bereit stellen. Sonst 
wird das nichts.

[Peter H. (borntopizza)]

Also die Seite aktualisiert sich ja von selbst, 1 mal die Sekunde.

Dann funktioniert ja die ganze Idee vom "Hacker" nicht...

Quelltext müsste oben sein, einmal vom server, und einmal vom Client.

[Peter H. (borntopizza)]

Also ich denke immernoch dass das an der char "reply" am Empfänger 
liegt. Weil am PC lässt sich die seite ja darstellen und alle GPIO Pins 
am Server lassen sich ja schalten, Temperatur wird auch dargestellt.

Müsste man dann die Website am Client auf "2 mal einlesen"?

[Einer K. (Gast)]

Die Meldung ist eine des Compilers, sie stammt nicht aus dem 
Protokollstack.

[Stefan F. (Gast)]

Waren Abblock-Kondensatoren gerade Mangelware?
An die Reset Pins gehören auch Kondensatoren.
Aref gehört nicht mit VCC kurzgeschlossen. Lies die entsprechenden 
Hinweise im Datenblatt!

Was funktioniert jetzt nicht, die Darstellung der Webseite im Browser 
oder das Schalten des Relais im Empfänger? Wie kommunizieren die beiden 
Mikrocontroller miteinander?

Inwiefern weicht die fehlerhafte Kommunikation vom Soll ab? Oder ist die 
Kommunikation gar nicht gestört sondern es scheitert an etwas anderem?

Hast du eine Möglichkeit, Debug Meldungen auszugeben? Wenn nicht, dann 
solltest du das bei beiden Mikrocontrollern schleunigst hinzufügen. Und 
lerne, mit Wireshark umzugehen.

Wir sehen uns dann wieder, wenn du die Fehlerursache mit diesen Mitteln 
eingekreist hast. Bis dahin empfehle ich hier eine Sendepause für alle, 
denn wilde Vermutungen enden erfahrungsgemäss in wüsten Beschimpfungen.