[Plugins] P044_P1WifiGateway. shouldn't this be freed at some point in PLUGIN_EXIT?

[Barracuda09]

See link:

ESPEasy/src/_P044_P1WifiGateway.ino

Line 138 in 0977b7e

Plugin_044_serial_buf = (char *)malloc(P044_BUFFER_SIZE);

And maybe change this to new and delete?

[TD-er]

As we speak, I am updating this plugin to use the Plugin_task_data and indeed it needs to free the data.
I was even thinking about using a std::vector for this one.

[Barracuda09]

I was even thinking about using a std::vector for this one.

Yes if that would be possible, I would go for that as well.

[Barracuda09]

And maybe use an enum (class) for this:

ESPEasy/src/_P044_P1WifiGateway.ino

Line 20 in 0977b7e

#define P044_DISABLED 0

EDIT you can even do it like this:
enum (class) State : byte {
Disabled,
...
};

[TD-er]

I did change it to be a String, since there were already functions implemented to do just what was already available in the String class.
See: ef5f519

[TD-er]

The code for the P044_P1WifiGateway is reverted to some state before my fixes of this issue.
So I added a quick fix for this memory leak.
Later we will have a proper look at the code to see what was causing the issues with the ser2net plugin.

Anyway this issue is now fixed, so I will close it.