Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitb64c1a1

Browse files
authored
Merge branch 'esp8266:master' into patch-1
2 parents474dcd3 +685f2c9 commitb64c1a1

File tree

17 files changed

+185
-80
lines changed

17 files changed

+185
-80
lines changed

‎cores/esp8266/Stream.h‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,21 +196,25 @@ class Stream: public Print {
196196
// returns number of transferred bytes
197197
size_tsendAvailable (Stream* to) {returnsendGeneric(to, -1, -1, oneShotMs::alwaysExpired); }
198198
size_tsendAvailable (Stream& to) {returnsendAvailable(&to); }
199+
size_tsendAvailable (Stream&& to) {returnsendAvailable(&to); }
199200

200201
// transfers data until timeout
201202
// returns number of transferred bytes
202203
size_tsendAll (Stream* to,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendGeneric(to, -1, -1, timeoutMs); }
203204
size_tsendAll (Stream& to,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendAll(&to, timeoutMs); }
205+
size_tsendAll (Stream&& to,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendAll(&to, timeoutMs); }
204206

205207
// transfers data until a char is encountered (the char is swallowed but not transferred) with timeout
206208
// returns number of transferred bytes
207209
size_tsendUntil (Stream* to,constint readUntilChar,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendGeneric(to, -1, readUntilChar, timeoutMs); }
208210
size_tsendUntil (Stream& to,constint readUntilChar,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendUntil(&to, readUntilChar, timeoutMs); }
211+
size_tsendUntil (Stream&& to,constint readUntilChar,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendUntil(&to, readUntilChar, timeoutMs); }
209212

210213
// transfers data until requested size or timeout
211214
// returns number of transferred bytes
212215
size_tsendSize (Stream* to,constssize_t maxLen,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendGeneric(to, maxLen, -1, timeoutMs); }
213216
size_tsendSize (Stream& to,constssize_t maxLen,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendSize(&to, maxLen, timeoutMs); }
217+
size_tsendSize (Stream&& to,constssize_t maxLen,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires) {returnsendSize(&to, maxLen, timeoutMs); }
214218

215219
StringreadStreamString (constssize_t maxLen = -1 ,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires);
216220
StringreadStreamStringUntil (constint readUntilChar,const oneShotMs::timeType timeoutMs = oneShotMs::neverExpires);

‎cores/esp8266/StreamString.h‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
#include"WString.h"
3030

3131
///////////////////////////////////////////////////////////////
32-
// S2Stream points to a String and makes it a Stream
32+
// S2Stream("String to Stream")points to a String and makes it a Stream
3333
// (it is also the helper for StreamString)
3434

3535
classS2Stream:publicStream
@@ -184,26 +184,26 @@ class S2Stream: public Stream
184184
return peekPointer <0 ? string->length() : string->length() - peekPointer;
185185
}
186186

187-
// calling setConsume() willconsume bytesas the stream is read
188-
// (enabled bydefault)
187+
// calling setConsume() willmake the string consumedas the stream is read.
188+
// (default behaviour)
189189
voidsetConsume()
190190
{
191191
peekPointer = -1;
192192
}
193193

194-
// Reading this stream will mark the string as read without consuming
195-
// (not enabled by default)
196-
// Calling resetPointer() resets the read state and allows rereading.
197-
voidresetPointer(int pointer =0)
194+
// Calling resetPointer() resets the read cursor and allows rereading.
195+
// (this is the opposite of default mode set by setConsume())
196+
voidresetPointer(size_t pointer =0)
198197
{
199-
peekPointer = pointer;
198+
peekPointer =std::min(pointer, (size_t)string->length());
200199
}
201200

202201
protected:
203202
String* string;
204203
int peekPointer;// -1:String is consumed / >=0:resettable pointer
205204
};
206205

206+
///////////////////////////////////////////////////////////////
207207
// StreamString is a S2Stream holding the String
208208

209209
classStreamString:publicString,publicS2Stream

‎cores/esp8266/Updater.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ bool UpdaterClass::end(bool evenIfRemaining){
284284
_hash->begin();
285285
for (uint32_t offset =0; offset < binSize; offset +=sizeof(buff)) {
286286
auto len =std::min(sizeof(buff), binSize - offset);
287-
ESP.flashRead(_startAddress + offset,reinterpret_cast<uint32_t *>(&buff[0]), len);
287+
ESP.flashRead(_startAddress + offset, buff, len);
288288
_hash->add(buff, len);
289289
}
290290
_hash->end();

‎cores/esp8266/WString.h‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
#include<utility>
3434
#include<type_traits>
3535

36-
// an abstract class used as a means toproide a unique pointer type
36+
// an abstract class used as a means toprovide a unique pointer type
3737
// but really has no body
3838
class__FlashStringHelper;
3939
#defineFPSTR(pstr_pointer) (reinterpret_cast<const __FlashStringHelper *>(pstr_pointer))
@@ -204,7 +204,7 @@ class String {
204204
boolconcat(double num);
205205

206206
// if there's not enough memory for the concatenated value, the string
207-
// will be left unchanged (but this isn'tsignalled in any way)
207+
// will be left unchanged (but this isn'tsignaled in any way)
208208
template<typename T>
209209
String &operator +=(const T &rhs) {
210210
concat(rhs);
@@ -343,7 +343,7 @@ class String {
343343
char *wbuffer() {returnconst_cast<char *>(buffer()); }// Writable version of buffer
344344

345345
// concatenation is done via non-member functions
346-
// make sure we still have access to internal methods, since we optimize based on capacity of both sides and want to manipulate internal buffers directly
346+
// make sure we still have access to internal methods, since we optimize based onthecapacity of both sides and want to manipulate internal buffers directly
347347
friend Stringoperator +(const String &lhs, String &&rhs);
348348
friend Stringoperator +(String &&lhs, String &&rhs);
349349
friend Stringoperator +(char lhs, String &&rhs);

‎cores/esp8266/cont.S‎

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,14 @@
2626
cont_suspend:
2727
/* a1: sp */
2828
/* a2: void* cont_ctx */
29-
/* adjust stackand save registers*/
29+
/* adjust stack */
3030
addia1,a1,-24
31+
32+
/* make sure that a1 points after cont_ctx.stack[] */
33+
addia4,a2,32
34+
bltua1,a4, cont_overflow
35+
36+
/* save registers */
3137
s32i a12,a1,0
3238
s32i a13,a1,4
3339
s32i a14,a1,8
@@ -47,6 +53,11 @@ cont_suspend:
4753
l32ia1,a2,4
4854
jxa0
4955

56+
cont_overflow:
57+
mov.na3,a1
58+
movia4, __stack_overflow
59+
jxa4
60+
5061
cont_continue:
5162
l32i a12,a1,0
5263
l32i a13,a1,4
@@ -113,20 +124,23 @@ cont_run:
113124
bneza4, cont_resume
114125
/* else */
115126
/* set new stack*/
116-
l32ia1,a2,16;
127+
l32ia1,a2,16
117128
/* goto pfn */
118129
movia2, cont_wrapper
119130
jxa2
120131

121132
cont_resume:
122133
/* a1 <- cont_ctx.sp_suspend */
123134
l32ia1,a2,12
135+
/* make sure that a1 points after cont_ctx.stack[] */
136+
addia5,a2,32
137+
bltua1,a5, cont_overflow
124138
/* reset yield flag, 0 -> cont_ctx.pc_suspend */
125139
movia3,0
126140
s32ia3,a2,8
127141
/* jump to saved cont_ctx.pc_suspend */
128142
movia0, cont_ret
129-
jxa4
143+
jxa4
130144

131145
cont_norm:
132146
/* calculate pointer to cont_ctx.struct_start from sp */

‎cores/esp8266/cont.h‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,16 @@
2222
#defineCONT_H_
2323

2424
#include<stdbool.h>
25+
#include<stdint.h>
2526

2627
#ifndefCONT_STACKSIZE
2728
#defineCONT_STACKSIZE 4096
2829
#endif
2930

31+
#ifndefCONT_STACKGUARD
32+
#defineCONT_STACKGUARD 0xfeefeffe
33+
#endif
34+
3035
#ifdef__cplusplus
3136
extern"C" {
3237
#endif
@@ -62,8 +67,11 @@ void cont_run(cont_t*, void (*pfn)(void));
6267
// execution state (registers and stack)
6368
voidcont_suspend(cont_t*);
6469

70+
// Check that cont resume state is valid. Immediately panics on failure.
71+
voidcont_check_overflow(cont_t*);
72+
6573
// Check guard bytes around the stack. Immediately panics on failure.
66-
voidcont_check(cont_t*);
74+
voidcont_check_guard(cont_t*);
6775

6876
// Go through stack and check how many bytes are most probably still unchanged
6977
// and thus weren't used by the user code. i.e. that stack space is free. (high water mark)
@@ -78,7 +86,6 @@ bool cont_can_suspend(cont_t* cont);
7886
// free, running the routine, then checking the max free
7987
voidcont_repaint_stack(cont_t*cont);
8088

81-
8289
#ifdef__cplusplus
8390
}
8491
#endif

‎cores/esp8266/cont_util.cpp‎

Lines changed: 17 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,38 +23,45 @@
2323
#include<stddef.h>
2424
#include<string.h>
2525

26-
#include"cont.h"
26+
#include"core_esp8266_features.h"
2727
#include"debug.h"
2828

29+
#include"cont.h"
30+
2931
extern"C"
3032
{
3133

32-
staticconstexprunsignedint CONT_STACKGUARD {0xfeefeffe };
34+
staticconstexpruint32_t CONT_STACKSIZE_U32 {sizeof(cont_t::stack) /sizeof(*cont_t::stack) };
3335

3436
voidcont_init(cont_t* cont) {
3537
memset(cont,0,sizeof(cont_t));
3638

3739
cont->stack_guard1 = CONT_STACKGUARD;
3840
cont->stack_guard2 = CONT_STACKGUARD;
39-
cont->stack_end = cont->stack +(sizeof(cont->stack) /4);
41+
cont->stack_end =&cont->stack[0] +CONT_STACKSIZE_U32;
4042
cont->struct_start = (unsigned*) cont;
4143

4244
// fill stack with magic values to check high water mark
43-
for(int pos =0; pos < (int)(sizeof(cont->stack) /4); pos++)
45+
for(int pos =0; pos < (int)(CONT_STACKSIZE_U32); pos++)
4446
{
4547
cont->stack[pos] = CONT_STACKGUARD;
4648
}
4749
}
4850

49-
void IRAM_ATTRcont_check(cont_t* cont) {
50-
if ((cont->stack_guard1== CONT_STACKGUARD)
51-
&& (cont->stack_guard2== CONT_STACKGUARD))
51+
void IRAM_ATTRcont_check_guard(cont_t* cont) {
52+
if ((cont->stack_guard1!= CONT_STACKGUARD)
53+
|| (cont->stack_guard2!= CONT_STACKGUARD))
5254
{
53-
return;
55+
__stack_chk_fail();
56+
__builtin_unreachable();
5457
}
58+
}
5559

56-
__stack_chk_fail();
57-
__builtin_unreachable();
60+
void IRAM_ATTRcont_check_overflow(cont_t* cont) {
61+
if (cont->sp_suspend && (cont->sp_suspend < &cont->stack[0])) {
62+
__stack_overflow(cont, cont->sp_suspend);
63+
__builtin_unreachable();
64+
}
5865
}
5966

6067
// No need for this to be in IRAM, not expected to be IRQ called

‎cores/esp8266/core_esp8266_main.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,7 @@ static void loop_wrapper() {
261261
}
262262
loop();
263263
loop_end();
264-
cont_check(g_pcont);
264+
cont_check_guard(g_pcont);
265265
if (serialEventRun) {
266266
serialEventRun();
267267
}

‎cores/esp8266/core_esp8266_postmortem.cpp‎

Lines changed: 49 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ static const char* s_unhandled_exception = NULL;
4848

4949
// Common way to notify about where the stack smashing happened
5050
// (but, **only** if caller uses our handler function)
51-
staticuint32_ts_stacksmash_addr =0;
51+
staticuint32_ts_stack_chk_addr =0;
5252

5353
voidabort() __attribute__((noreturn));
5454
staticvoiduart_write_char_d(char c);
@@ -59,6 +59,7 @@ static void print_stack(uint32_t start, uint32_t end);
5959
// using numbers different from "REASON_" in user_interface.h (=0..6)
6060
enum rst_reason_sw
6161
{
62+
REASON_USER_STACK_OVERFLOW =252,
6263
REASON_USER_STACK_SMASH =253,
6364
REASON_USER_SWEXCEPTION_RST =254
6465
};
@@ -188,7 +189,7 @@ static void postmortem_report(uint32_t sp_dump) {
188189
}
189190
elseif (rst_info.reason == REASON_SOFT_WDT_RST) {
190191
ets_printf_P(PSTR("\nSoft WDT reset"));
191-
constchar infinite_loop[] = {0x06,0xff,0xff };// loop: j loop
192+
constuint8_t infinite_loop[] = {0x06,0xff,0xff };// loop: j loop
192193
if (is_pc_valid(rst_info.epc1) &&0 ==memcmp_P(infinite_loop, (PGM_VOID_P)rst_info.epc1,3u)) {
193194
// The SDK is riddled with these. They are usually preceded by an ets_printf.
194195
ets_printf_P(PSTR(" - deliberate infinite loop detected"));
@@ -198,17 +199,23 @@ static void postmortem_report(uint32_t sp_dump) {
198199
rst_info.exccause,/* Address executing at time of Soft WDT level-1 interrupt*/ rst_info.epc1,0,0,0,0);
199200
}
200201
elseif (rst_info.reason == REASON_USER_STACK_SMASH) {
201-
ets_printf_P(PSTR("\nStack smashing detected.\n"));
202-
ets_printf_P(PSTR("\nException (%d):\nepc1=0x%08x epc2=0x%08x epc3=0x%08x excvaddr=0x%08x depc=0x%08x\n"),
203-
5/* Alloca exception, closest thing to stack fault*/, s_stacksmash_addr,0,0,0,0);
204-
}
202+
ets_printf_P(PSTR("\nStack smashing detected at 0x%08x\n"), s_stack_chk_addr);
203+
}
204+
elseif (rst_info.reason == REASON_USER_STACK_OVERFLOW) {
205+
ets_printf_P(PSTR("\nStack overflow detected\n"));
206+
}
205207
else {
206208
ets_printf_P(PSTR("\nGeneric Reset\n"));
207209
}
208210

209-
uint32_t cont_stack_start = (uint32_t) &(g_pcont->stack);
210-
uint32_t cont_stack_end = (uint32_t) g_pcont->stack_end;
211-
uint32_t stack_end;
211+
uint32_t cont_stack_start;
212+
if (rst_info.reason == REASON_USER_STACK_SMASH) {
213+
cont_stack_start = s_stack_chk_addr;
214+
}else {
215+
cont_stack_start = (uint32_t) (&g_pcont->stack[0]);
216+
}
217+
218+
uint32_t cont_stack_end = cont_stack_start + CONT_STACKSIZE;
212219

213220
// amount of stack taken by interrupt or exception handler
214221
// and everything up to __wrap_system_restart_local
@@ -249,15 +256,21 @@ static void postmortem_report(uint32_t sp_dump) {
249256
sp_dump =stack_thunk_get_cont_sp();
250257
}
251258

252-
if (sp_dump > cont_stack_start && sp_dump < cont_stack_end) {
259+
uint32_t stack_end;
260+
261+
// above and inside of cont, dump from the sp to the bottom of the stack
262+
if ((rst_info.reason == REASON_USER_STACK_OVERFLOW)
263+
|| ((sp_dump > cont_stack_start) && (sp_dump < cont_stack_end)))
264+
{
253265
ets_printf_P(PSTR("\nctx: cont\n"));
254266
stack_end = cont_stack_end;
255267
}
268+
// in system, reposition to a known address
269+
// it's actually 0x3ffffff0, but the stuff below ets_run
270+
// is likely not really relevant to the crash
256271
else {
257272
ets_printf_P(PSTR("\nctx: sys\n"));
258273
stack_end =0x3fffffb0;
259-
// it's actually 0x3ffffff0, but the stuff below ets_run
260-
// is likely not really relevant to the crash
261274
}
262275

263276
ets_printf_P(PSTR("sp: %08x end: %08x offset: %04x\n"), sp_dump, stack_end, offset);
@@ -296,11 +309,20 @@ static void print_stack(uint32_t start, uint32_t end) {
296309
for (uint32_t pos = start; pos < end; pos +=0x10) {
297310
uint32_t* values = (uint32_t*)(pos);
298311

312+
// avoid printing irrelevant data
313+
if ((values[0] == CONT_STACKGUARD)
314+
&& (values[0] == values[1])
315+
&& (values[1] == values[2])
316+
&& (values[2] == values[3]))
317+
{
318+
continue;
319+
}
320+
299321
// rough indicator: stack frames usually have SP saved as the second word
300-
bool looksLikeStackFrame = (values[2] == pos +0x10);
322+
constbool looksLikeStackFrame = (values[2] == pos +0x10);
301323

302324
ets_printf_P(PSTR("%08x: %08x %08x %08x %08x %c\n"),
303-
pos, values[0], values[1], values[2], values[3], (looksLikeStackFrame)?'<':'');
325+
pos, values[0], values[1], values[2], values[3], (looksLikeStackFrame) ?'<' :'');
304326
}
305327
}
306328

@@ -370,7 +392,7 @@ void __panic_func(const char* file, int line, const char* func) {
370392
uintptr_t __stack_chk_guard =0x08675309 ^ RANDOM_REG32;
371393
void__stack_chk_fail(void) {
372394
s_user_reset_reason = REASON_USER_STACK_SMASH;
373-
s_stacksmash_addr = (uint32_t)__builtin_return_address(0);
395+
s_stack_chk_addr = (uint32_t)__builtin_return_address(0);
374396

375397
if (gdb_present())
376398
__asm____volatile__ ("syscall");// triggers GDB when enabled
@@ -382,4 +404,16 @@ void __stack_chk_fail(void) {
382404
__builtin_unreachable();// never reached, needed to satisfy "noreturn" attribute
383405
}
384406

407+
void__stack_overflow(cont_t* cont,uint32_t* sp) {
408+
s_user_reset_reason = REASON_USER_STACK_OVERFLOW;
409+
s_stack_chk_addr = (uint32_t)&cont->stack[0];
410+
411+
if (gdb_present())
412+
__asm____volatile__ ("syscall");// triggers GDB when enabled
413+
414+
postmortem_report((uint32_t)sp);
415+
416+
__builtin_unreachable();// never reached, needed to satisfy "noreturn" attribute
417+
}
418+
385419
}// extern "C"

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp