Lerking/micropython
1
0
Fork 0
mirror of https://github.com/micropython/micropython.git synced 2026-01-04 11:10:14 +01:00
Code Issues Actions 52 Packages Projects Releases Wiki Activity

extmod/moductypes: Error if small ints are not big enough.

Browse Source 
Signed-off-by: Jeff Epler <jepler@unpythonic.net>
This commit is contained in:
Jeff Epler
2025-09-20 11:48:23 -05:00
committed by Damien George
parent 0a41838bec
commit 2d08f2f631
1 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
extmod/moductypes.c
View File

@@ -28,6 +28,7 @@
#include <string.h> #include <string.h>
#include <stdint.h> #include <stdint.h>
#include "py/smallint.h"
#include "py/runtime.h" #include "py/runtime.h"
#include "py/objtuple.h" #include "py/objtuple.h"
#include "py/binary.h" #include "py/binary.h"
@@ -94,6 +95,17 @@ static MP_NORETURN void syntax_error(void) {
} }
static mp_obj_t uctypes_struct_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *args) { static mp_obj_t uctypes_struct_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *args) {
// Because mpy-cross turns an expression like `uctypes.INT8` into a single
// constant integer load, the uctypes constant values must be consistent, no
// matter the OBJ_REPR and mp_int_t type.
//
// However, these constants are 31 bits (counting the sign bit) while
// OBJ_REPR_B with 32-bit mp_int_t provides only 30 bits of small integer, so
// this combination is unsupported.
//
// For more information, see https://github.com/micropython/micropython/issues/18105
MP_STATIC_ASSERT(MP_SMALL_INT_BITS >= 31);
mp_arg_check_num(n_args, n_kw, 2, 3, false); mp_arg_check_num(n_args, n_kw, 2, 3, false);
mp_obj_uctypes_struct_t *o = mp_obj_malloc(mp_obj_uctypes_struct_t, type); mp_obj_uctypes_struct_t *o = mp_obj_malloc(mp_obj_uctypes_struct_t, type);
o->addr = (void *)(uintptr_t)mp_obj_get_int_truncated(args[0]); o->addr = (void *)(uintptr_t)mp_obj_get_int_truncated(args[0]);