Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ESPEasy runs out of memory when making too many connections. #253

Closed
psy0rz opened this issue May 1, 2017 · 1 comment
Closed

ESPEasy runs out of memory when making too many connections. #253

psy0rz opened this issue May 1, 2017 · 1 comment
Labels
Category: Core related Related to the (external) core libraries Type: Bug Considered a bug
Milestone

Comments

@psy0rz
Copy link
Member

psy0rz commented May 1, 2017

Steps to reproduce

  • Add a controler like C001 (domoticz http)
  • set messagedelay to 0
  • add a device that reports every second.
  • make sure the server does not honor the 'Connection: close' header, for example use nc:
    nc -k -l -v 8383

keep in mind this problem only occurs if we take the initiative to disconnect before the server does.

Expected behavior

Should make a new connection every second and close it without leaking memory.

Actual behavior

Everytime a new connection is started it leaks a bit of mem.

Actually a feature of wifi client is causing the problem: esp8266/Arduino#1923

@psy0rz psy0rz added Type: Bug Considered a bug Category: Core related Related to the (external) core libraries labels May 1, 2017
@psy0rz psy0rz added this to the 2.0.0 milestone May 1, 2017
@psy0rz psy0rz changed the title ESPEasy runs out of memory when making too many connectoins ESPEasy runs out of memory when making too many connections. May 1, 2017
@psy0rz
Copy link
Member Author

psy0rz commented May 1, 2017

looked for a long time in the core-libraries to find a clean non-hackish way to do this, but couldnt find it.

so we just do this for now:

  while(tcp_tw_pcbs!=NULL)
  {
    tcp_abort(tcp_tw_pcbs);
  }

psy0rz added a commit that referenced this issue May 1, 2017
@psy0rz psy0rz closed this as completed May 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Core related Related to the (external) core libraries Type: Bug Considered a bug
Projects
None yet
Development

No branches or pull requests

1 participant