Stowage/cpython
2
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-12-07 13:50:06 +00:00
Code Issues Projects Releases Packages Wiki Activity

gh-127635: Use flexible array in tracemalloc (#141991)

Browse source 
Replace frames[1] with frames[] in tracemalloc_traceback structure.
This commit is contained in:
Victor Stinner 2025-11-27 12:32:31 +01:00 • committed by GitHub
parent 7fe1a18b77
commit 83d8134c5b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 30 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

20
Include/internal/pycore_tracemalloc.h
View file

@ -30,8 +30,8 @@ struct _PyTraceMalloc_Config {
};
/* Pack the frame_t structure to reduce the memory footprint on 64-bit
architectures: 12 bytes instead of 16. */
/* Pack the tracemalloc_frame and tracemalloc_traceback structures to reduce
the memory footprint on 64-bit architectures: 12 bytes instead of 16. */
#if defined(_MSC_VER)
#pragma pack(push, 4)
#endif
@ -46,18 +46,22 @@ tracemalloc_frame {
PyObject *filename;
unsigned int lineno;
};
#ifdef _MSC_VER
#pragma pack(pop)
#endif
struct tracemalloc_traceback {
struct
#ifdef __GNUC__
__attribute__((packed))
#endif
tracemalloc_traceback {
Py_uhash_t hash;
/* Number of frames stored */
uint16_t nframe;
/* Total number of frames the traceback had */
uint16_t total_nframe;
struct tracemalloc_frame frames[1];
struct tracemalloc_frame frames[];
};
#ifdef _MSC_VER
#pragma pack(pop)
#endif
struct _tracemalloc_runtime_state {
@ -95,7 +99,7 @@ struct _tracemalloc_runtime_state {
Protected by TABLES_LOCK(). */
_Py_hashtable_t *domains;
struct tracemalloc_traceback empty_traceback;
struct tracemalloc_traceback *empty_traceback;
Py_tss_t reentrant_key;
};

26
Python/tracemalloc.c
View file

@ -46,7 +46,7 @@ typedef struct tracemalloc_frame frame_t;
typedef struct tracemalloc_traceback traceback_t;
#define TRACEBACK_SIZE(NFRAME) \
(sizeof(traceback_t) + sizeof(frame_t) * (NFRAME - 1))
(sizeof(traceback_t) + sizeof(frame_t) * (NFRAME))
static const int MAX_NFRAME = UINT16_MAX;
@ -329,8 +329,9 @@ traceback_new(void)
traceback->nframe = 0;
traceback->total_nframe = 0;
traceback_get_frames(traceback);
if (traceback->nframe == 0)
return &tracemalloc_empty_traceback;
if (traceback->nframe == 0) {
return tracemalloc_empty_traceback;
}
traceback->hash = traceback_hash(traceback);
/* intern the traceback */
@ -754,12 +755,18 @@ _PyTraceMalloc_Init(void)
return _PyStatus_NO_MEMORY();
}
tracemalloc_empty_traceback.nframe = 1;
tracemalloc_empty_traceback.total_nframe = 1;
assert(tracemalloc_empty_traceback == NULL);
tracemalloc_empty_traceback = raw_malloc(TRACEBACK_SIZE(1));
if (tracemalloc_empty_traceback == NULL) {
return _PyStatus_NO_MEMORY();
}
tracemalloc_empty_traceback->nframe = 1;
tracemalloc_empty_traceback->total_nframe = 1;
/* borrowed reference */
tracemalloc_empty_traceback.frames[0].filename = &_Py_STR(anon_unknown);
tracemalloc_empty_traceback.frames[0].lineno = 0;
tracemalloc_empty_traceback.hash = traceback_hash(&tracemalloc_empty_traceback);
tracemalloc_empty_traceback->frames[0].filename = &_Py_STR(anon_unknown);
tracemalloc_empty_traceback->frames[0].lineno = 0;
tracemalloc_empty_traceback->hash = traceback_hash(tracemalloc_empty_traceback);
tracemalloc_config.initialized = TRACEMALLOC_INITIALIZED;
return _PyStatus_OK();
@ -782,6 +789,9 @@ tracemalloc_deinit(void)
_Py_hashtable_destroy(tracemalloc_filenames);
PyThread_tss_delete(&tracemalloc_reentrant_key);
raw_free(tracemalloc_empty_traceback);
tracemalloc_empty_traceback = NULL;
}