See the logs below. It seems that after some time the message was finally deleted. However, we should handle this better.

BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETING FAILED
BUFFER DELETED

In order to be included in Arduino Library Manager, we must adhere to their specification.

@Ales-Svoboda Finish implementation of base functionality and then I will help you to make necessary changes in order to make the code compatible.

When user tries to set up a parameter longer than 16 characters (maximum supported length), and also he wants to set a callback, Adeon code is accessing unallocated object.

1. First, Adeon tries to store the parameter in the list
2. Then it compares the parameter length, and because it exceeds maximum supported length, it won't store the parameter
3. The function returns NULL reference
4. Adeon doesn't check the NULL reference, and stores pointer to callback function
5. Adeon crashes

This issue was fixed in #10

.pio/libdeps/m5stack-core-esp32/Adeon/src/AdeonGSM.cpp: In member function 'bool Adeon::Parser::isHashParsingValid()':
.pio/libdeps/m5stack-core-esp32/Adeon/src/AdeonGSM.cpp:358:25: warning: array subscript is above array bounds [-Warray-bounds]
_tmpHash[hashLen] = _nullChar;

At this moment, we can register a callback for each parameter. After it is set, we can do some action. However, in many cases we send a group of parameters, and it makes sense to do some stuff only once, e.g., notify an user that his commands were received.

We should introduce a new callback for this purpose.

Adeon tries to access invalid memory during hash validation. PR #10 disables the code, and verifies only that the message hash has correct length. At least it works.

@Robert-Faltus please take a look at it.

• Arduino board: UNO R3

• Arduino IDE version (found in Arduino -> About Arduino menu): 1.8.12

• PlatformIO Core version, if you use PlatformIO (found in VS Code tab): INSERT
  PLATFORMIO CORE VERSION HERE

• List the steps to reproduce the problem below (if possible attach a sketch or
  copy the sketch code in too):

Compile example from Arduino IDE in File -> Examples -> Adeon -> SIMComGsm.ino

C:\Arduino\libraries\Adeon\src\utility\MD5.cpp: In static member function 'static void MD5::MD5Update(void*, const void*, size_t)':

C:\Arduino\libraries\Adeon\src\utility\MD5.cpp:215:21: warning: right shift count >= width of type [-Wshift-count-overflow]

  ctx->hi += size >> 29;

                     ^~

While I worked on GPRS support, I noticed that all examples and libraries use phone number with international extension, including the + sign. Adeon doesn't use this sign, which makes it incompatible with the rest.

A phone number starting with a country extension without a plus sign, e.g., 420xxxxxxxxx, is invalid. A valid format is either "+420xxxxxxxxx" or "00420xxxxxxxxx".

As a bigger refactor, we should rework Adeon to support Stream interface. This would allow us to plug Adeon easily into other network libraries, including TinyGSM.

I gave Adeon SIMlib a reference to Serial1, and yet it was also affecting other device on Serial2. After I modified (and simplified) the code, everything worked fine.

Also, HardwareSerial and SoftwareSerial inherit from the Stream class. The code can be heavily simplified.

Following error is thrown while building sketches for Arduino Mega:

/home/travis/Arduino/libraries/Adeon/src/utility/MD5.cpp: In static member function 'static void MD5::MD5Update(void*, const void*, size_t)':

/home/travis/Arduino/libraries/Adeon/src/utility/MD5.cpp:215:21: warning: right shift count >= width of type [-Wshift-count-overflow]

  ctx->hi += size >> 29

Note:
This applies to GSM modules which are officialy supported by Adeon library (eg. SimCOM modules).

Description:
Add support for sending status info via SMS after receiving of the Adeon command or phone call with sendFeedback(char* msg). This status info should be sent to sender/caller only if the sender is authorized with ADEON_ADMIN or ADEON_USER user rights (ADEON_HOST excluded).

Example of status info:

• Command accepted.
• Command declined – user is not authorized.
• Command declined – wrong format.
• Drop-call accepted.
• Drop-call declined.

Example: SIMComGSM

When I tried the example for the first few times, everything worked. But after the module received more messages, it started crashing every time a new message arrived. I think the responsible code for the crash is in Adeon.

I "solved" the problem by removing all messages from the module. However, this solution just confirmed what's the problem. We need to find a fix.