Fixed bug that incremented _smsDataIndex in peek, clipping the first letter of a incoming message

Patrick-McGuire · Patrick-McGuire · commit de1044670c73 · 2022-06-01T16:19:25.000-04:00
diff --git a/src/NB_SMS.cpp b/src/NB_SMS.cpp
@@ -394,7 +394,7 @@ int NB_SMS::peek()
     return *_ptrUTF8;
   }
   if (_smsDataIndex < (signed)_incomingBuffer.length() && _smsDataIndex <= _smsDataEndIndex) {
-    char c = _incomingBuffer[_smsDataIndex++];
+    char c = _incomingBuffer[_smsDataIndex+1];
     if (_charset == SMS_CHARSET_GSM
         && (c >= 0x80 || c <= 0x24 || (c&0x1F) == 0 || (c&0x1F) >= 0x1B)) {
       for (auto &gsmchar : _gsmUTF8map) {

Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,7 @@ int NB_SMS::peek()`
`394`	`394`	`return *_ptrUTF8;`
`395`	`395`	`}`
`396`	`396`	`if (_smsDataIndex < (signed)_incomingBuffer.length() && _smsDataIndex <= _smsDataEndIndex) {`
`397`		`- char c = _incomingBuffer[_smsDataIndex++];`
	`397`	`+ char c = _incomingBuffer[_smsDataIndex+1];`
`398`	`398`	`if (_charset == SMS_CHARSET_GSM`
`399`	`399`	`&& (c >= 0x80 \|\| c <= 0x24 \|\| (c&0x1F) == 0 \|\| (c&0x1F) >= 0x1B)) {`
`400`	`400`	`for (auto &gsmchar : _gsmUTF8map) {`