Skip to content

Commit dd8c2e6

Browse files
authored
Fixed OOM double count in umm_realloc. (#7768)
1 parent 9eb618e commit dd8c2e6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

cores/esp8266/umm_malloc/umm_malloc.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -1038,7 +1038,7 @@ void *umm_realloc( void *ptr, size_t size ) {
10381038
} else {
10391039
DBGLOG_DEBUG( "realloc %i to a bigger block %i failed - return NULL and leave the old block!\n", blockSize, blocks );
10401040
/* This space intentionally left blnk */
1041-
STATS__OOM_UPDATE();
1041+
/* STATS__OOM_UPDATE() has already been called by umm_malloc_core - don't duplicate count */
10421042
}
10431043
/* This is not accurate for OOM case; however, it will work for
10441044
* stopping a call to free before return.
@@ -1117,7 +1117,7 @@ void *umm_realloc( void *ptr, size_t size ) {
11171117
} else {
11181118
DBGLOG_DEBUG( "realloc %d to a bigger block %d failed - return NULL and leave the old block!\n", blockSize, blocks );
11191119
/* This space intentionally left blnk */
1120-
STATS__OOM_UPDATE();
1120+
/* STATS__OOM_UPDATE() has already been called by umm_malloc_core - don't duplicate count */
11211121
}
11221122
/* This is not accurate for OOM case; however, it will work for
11231123
* stopping a call to free before return.
@@ -1142,7 +1142,7 @@ void *umm_realloc( void *ptr, size_t size ) {
11421142
} else {
11431143
DBGLOG_DEBUG( "realloc %d to a bigger block %d failed - return NULL and leave the old block!\n", blockSize, blocks );
11441144
/* This space intentionally left blnk */
1145-
STATS__OOM_UPDATE();
1145+
/* STATS__OOM_UPDATE() has already been called by umm_malloc_core - don't duplicate count */
11461146
}
11471147
/* This is not accurate for OOM case; however, it will work for
11481148
* stopping a call to free before return.

0 commit comments

Comments
 (0)