annotate components/uart_b_lib @ 516:1ed9de6c90bd

src/g23m-gsm/sms/sms_for.c: bogus malloc removed The new error handling code that was not present in TCS211 blob version contains a malloc call that is bogus for 3 reasons: 1) The memory allocation in question is not needed in the first place; 2) libc malloc is used instead of one of the firmware's proper ways; 3) The memory allocation is made inside a function and then never freed, i.e., a memory leak. This bug was caught in gcc-built FreeCalypso fw projects (Citrine and Selenite) because our gcc environment does not allow any use of libc malloc (any reference to malloc produces a link failure), but this code from TCS3.2 is wrong even for Magnetite: if this code path is executed repeatedly over a long time, the many small allocations made by this malloc call without a subsequent free will eventually exhaust the malloc heap provided by the TMS470 environment, malloc will start returning NULL, and the bogus code will treat it as an error. Because the memory allocation in question is not needed at all, the fix entails simply removing it.
author Mychaela Falconia <falcon@freecalypso.org>
date Sun, 22 Jul 2018 06:04:49 +0000
parents ad7f986afae3
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
173
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
1 # Building uart_b_lib.lib from the TCS3.2 source
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
2
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
3 CFLAGS="-me -mt -o -pw2 -x -mw"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
4
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
5 # Defines
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
6
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
7 CPPFLAGS=-DCCDTABLES_EXTERN
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
8 CPPFLAGS="$CPPFLAGS -DOPTION_MULTITHREAD"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
9 CPPFLAGS="$CPPFLAGS -DNEW_FRAME"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
10 CPPFLAGS="$CPPFLAGS -DNEW_ENTITY"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
11 CPPFLAGS="$CPPFLAGS -D_TARGET_"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
12 CPPFLAGS="$CPPFLAGS -DSHARED_VSI"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
13 CPPFLAGS="$CPPFLAGS -DALR"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
14 CPPFLAGS="$CPPFLAGS -DFF_HOMEZONE"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
15 CPPFLAGS="$CPPFLAGS -DFF_MMI_SAT_ICON"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
16 CPPFLAGS="$CPPFLAGS -DFF_MMI_SMS_DYNAMIC"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
17 CPPFLAGS="$CPPFLAGS -DFF_CPHS"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
18 CPPFLAGS="$CPPFLAGS -D_TMS470"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
19 CPPFLAGS="$CPPFLAGS -DUART"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
20 CPPFLAGS="$CPPFLAGS -DFF_ATI"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
21 CPPFLAGS="$CPPFLAGS -DNWARN"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
22 CPPFLAGS="$CPPFLAGS -DFF_EM_MODE"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
23 CPPFLAGS="$CPPFLAGS -DAT_INTERPRETER"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
24 CPPFLAGS="$CPPFLAGS -DUSE_L1FD_FUNC_INTERFACE"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
25
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
26 if [ "$SRVC" != 0 ]
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
27 then
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
28 CPPFLAGS="$CPPFLAGS -DFAX_AND_DATA"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
29 if [ "$SRVC" = 1 ]
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
30 then
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
31 CPPFLAGS="$CPPFLAGS -DFF_FAX"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
32 fi
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
33 CPPFLAGS="$CPPFLAGS -DUSE_L2FD_FUNC_INTERFACE"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
34 fi
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
35
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
36 CPPFLAGS="$CPPFLAGS -DSRVC=$SRVC"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
37 CPPFLAGS="$CPPFLAGS -DMMI=$MMI"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
38
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
39 if [ "$MMI" = 0 ]
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
40 then
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
41 CPPFLAGS="$CPPFLAGS -DACI"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
42 elif [ "$MMI" = 2 ]
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
43 then
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
44 CPPFLAGS="$CPPFLAGS -DMFW"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
45 fi
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
46
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
47 CPPFLAGS="$CPPFLAGS -DPHONEBOOK_EXTENSION"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
48
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
49 if [ "$GPRS" = 1 ]
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
50 then
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
51 CPPFLAGS="$CPPFLAGS -DGPRS"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
52 fi
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
53
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
54 CPPFLAGS="$CPPFLAGS -DDTI2"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
55 CPPFLAGS="$CPPFLAGS -DDTILIB"
482
ad7f986afae3 pass -DCHIPSET and -DBOARD for all TCS3.2 components
Mychaela Falconia <falcon@freecalypso.org>
parents: 173
diff changeset
56 CPPFLAGS="$CPPFLAGS -DCHIPSET=$CHIPSET"
ad7f986afae3 pass -DCHIPSET and -DBOARD for all TCS3.2 components
Mychaela Falconia <falcon@freecalypso.org>
parents: 173
diff changeset
57 CPPFLAGS="$CPPFLAGS -DBOARD=41"
173
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
58
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
59 # Includes
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
60
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
61 SRCDIR=$SRC/g23m-aci/uart
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
62
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
63 CPPFLAGS="$CPPFLAGS -I$SRC/$CONDAT/com/inc"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
64 CPPFLAGS="$CPPFLAGS -I$SRC/$GPF/inc"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
65 CPPFLAGS="$CPPFLAGS -I$SRC/$CONDAT/com/include"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
66 CPPFLAGS="$CPPFLAGS -I../../$CDGINC"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
67 CPPFLAGS="$CPPFLAGS -I$SRC/$GPF/inc/nuc/arm7"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
68 CPPFLAGS="$CPPFLAGS -I$SRCDIR"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
69 CPPFLAGS="$CPPFLAGS -I$SRC/cs/layer1/cust0"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
70 CPPFLAGS="$CPPFLAGS -I.."
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
71 CPPFLAGS="$CPPFLAGS -I../config"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
72 CPPFLAGS="$CPPFLAGS -I$SRC/cs/riviera"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
73 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_app"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
74 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
75 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/abb"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
76 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/armio"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
77 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/clkm"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
78 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/conf"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
79 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/dma"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
80 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/dsp_dwnld"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
81 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/inth"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
82 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/memif"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
83 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/rhea"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
84 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/security"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
85 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/spi"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
86 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/timer"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
87 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/uart"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
88 CPPFLAGS="$CPPFLAGS -I$SRC/cs/drivers/drv_core/ulpd"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
89 CPPFLAGS="$CPPFLAGS -I$SRC/cs/system"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
90 CPPFLAGS="$CPPFLAGS -I$SRC/cs/layer1/include"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
91 CPPFLAGS="$CPPFLAGS -I$SRC/cs/services"
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
92
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
93 # Source modules
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
94
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
95 cfile_str2ind $SRCDIR/uart_kerf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
96 cfile_str2ind $SRCDIR/uart_kerp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
97 cfile_str2ind $SRCDIR/uart_kers.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
98 cfile_str2ind $SRCDIR/uart_drxf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
99 cfile_str2ind $SRCDIR/uart_drxp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
100 cfile_str2ind $SRCDIR/uart_drxs.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
101 cfile_str2ind $SRCDIR/uart_dtxf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
102 cfile_str2ind $SRCDIR/uart_dtxp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
103 cfile_str2ind $SRCDIR/uart_dtxs.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
104 cfile_str2ind $SRCDIR/uart_rxf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
105 cfile_str2ind $SRCDIR/uart_rxp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
106 cfile_str2ind $SRCDIR/uart_rxs.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
107 cfile_str2ind $SRCDIR/uart_txf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
108 cfile_str2ind $SRCDIR/uart_txp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
109 cfile_str2ind $SRCDIR/uart_txs.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
110 cfile_str2ind $SRCDIR/uart_rtf.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
111 cfile_str2ind $SRCDIR/uart_rtp.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
112 cfile_str2ind $SRCDIR/uart_rts.c
bf64d785238a uart_b_lib.lib compiles
Mychaela Falconia <falcon@freecalypso.org>
parents:
diff changeset
113 cfile_str2ind $SRCDIR/uart_pei.c