HAL版例程USMART判断参数个数问题

elecrui

USMART例程下usmart_str.c文件中usmart_get_fname函数原文件第390-410行（下文代码120行-140行），在接收完函数名后计算参数个数，是通过括号中“,”个数计算的，但这里并未判断是否正在字符串内，如果参数中包含字符串，且字符串中有“,”，可能造成参数个数计算错误。

1. /**
   
2. * @brief       从str中得到函数名
   
3. * [url=home.php?mod=space&uid=271674]@param[/url]       str   : 源字符串指针
   
4. * @param       fname : 获取到的函数名字指针
   
5. * @param       pnum  : 函数的参数个数
   
6. * @param       rval  : 是否需要显示返回值(0,不需要;1,需要)
   
7. * @retval      0,成功;其他,错误代码.
   
8. */
   
9. uint8_t usmart_get_fname(char *str, char *fname, uint8_t *pnum, uint8_t *rval)
   
10. {
    
11.     uint8_t res;
    
12.     uint8_t fover = 0;  /* 括号深度 */
    
13.     char *strtemp;
    
14.     uint8_t offset = 0;
    
15.     uint8_t parmnum = 0;
    
16.     uint8_t temp = 1;
    
17.     char fpname[6];  /* void+X+'/0' */
    
18.     uint8_t fplcnt = 0; /* 第一个参数的长度计数器 */
    
19.     uint8_t pcnt = 0;   /* 参数计数器 */
    
20.     uint8_t nchar;
    
21.     /* 判断函数是否有返回值 */
    
22.     strtemp = str;
    
23. 
    
24.     while (*strtemp != '\0')    /* 没有结束 */
    
25.     {
    
26.         if (*strtemp != ' ' && (pcnt & 0X7F) < 5)   /* 最多记录5个字符 */
    
27.         {
    
28.             if (pcnt == 0)pcnt |= 0X80; /* 置位最高位,标记开始接收返回值类型 */
    
29. 
    
30.             if (((pcnt & 0x7f) == 4) && (*strtemp != '*'))break;    /* 最后一个字符,必须是* */
    
31. 
    
32.             fpname[pcnt & 0x7f] = *strtemp; /* 记录函数的返回值类型 */
    
33.             pcnt++;
    
34.         }
    
35.         else if (pcnt == 0X85)
    
36.         {
    
37.             break;
    
38.         }
    
39. 
    
40.         strtemp++;
    
41.     }
    
42. 
    
43.     if (pcnt)   /* 接收完了 */
    
44.     {
    
45.         fpname[pcnt & 0x7f] = '\0'; /* 加入结束符 */
    
46. 
    
47.         if (usmart_strcmp(fpname, "void") == 0)
    
48.         {
    
49.             *rval = 0;  /* 不需要返回值 */
    
50.         }
    
51.         else
    
52.         {
    
53.             *rval = 1;  /* 需要返回值 */
    
54.         }
    
55. 
    
56.         pcnt = 0;
    
57.     }
    
58. 
    
59.     res = 0;
    
60.     strtemp = str;
    
61. 
    
62.     while (*strtemp != '(' && *strtemp != '\0')   /* 此代码找到函数名的真正起始位置 */
    
63.     {
    
64.         strtemp++;
    
65.         res++;
    
66. 
    
67.         if (*strtemp == ' ' || *strtemp == '*')
    
68.         {
    
69.             nchar = usmart_search_nextc(strtemp);   /* 获取下一个字符 */
    
70. 
    
71.             if (nchar != '(' && nchar != '*')offset = res;  /* 跳过空格和*号 */
    
72.         }
    
73.     }
    
74. 
    
75.     strtemp = str;
    
76. 
    
77.     if (offset)strtemp += offset + 1;   /* 跳到函数名开始的地方 */
    
78. 
    
79.     res = 0;
    
80.     nchar = 0;  /* 是否正在字符串里面的标志,0，不在字符串;1，在字符串; */
    
81. 
    
82.     while (1)
    
83.     {
    
84.         if (*strtemp == 0)
    
85.         {
    
86.             res = USMART_FUNCERR;   /* 函数错误 */
    
87.             break;
    
88.         }
    
89.         else if (*strtemp == '(' && nchar == 0)
    
90.         {
    
91.             fover++;    /* 括号深度增加一级 */
    
92.         }
    
93.         else if (*strtemp == ')' && nchar == 0)
    
94.         {
    
95.             if (fover)
    
96.             {
    
97.                 fover--;
    
98.             }
    
99.             else
    
100.             {
     
101.                 res = USMART_FUNCERR;  /* 错误结束,没收到'(' */
     
102.             }
     
103. 
     
104.             if (fover == 0)break;       /* 到末尾了,退出 */
     
105.         }
     
106.         else if (*strtemp == '"')
     
107.         {
     
108.             nchar = !nchar;
     
109.         }
     
110. 
     
111.         if (fover == 0)   /* 函数名还没接收完 */
     
112.         {
     
113.             if (*strtemp != ' ')    /* 空格不属于函数名 */
     
114.             {
     
115.                 *fname = *strtemp;  /* 得到函数名 */
     
116.                 fname++;
     
117.             }
     
118.         }
     
119.         else     /* 已经接受完了函数名了. */
     
120.         {
     
121.             if (*strtemp == ',')
     
122.             {
     
123.                 temp = 1;           /* 使能增加一个参数 */
     
124.                 pcnt++;
     
125.             }
     
126.             else if (*strtemp != ' ' && *strtemp != '(')
     
127.             {
     
128.                 if (pcnt == 0 && fplcnt < 5)    /* 当第一个参数来时,为了避免统计void类型的参数,必须做判断. */
     
129.                 {
     
130.                     fpname[fplcnt] = *strtemp;  /* 记录参数特征. */
     
131.                     fplcnt++;
     
132.                 }
     
133. 
     
134.                 temp++;     /* 得到有效参数(非空格) */
     
135.             }
     
136. 
     
137.             if (fover == 1 && temp == 2)
     
138.             {
     
139.                 temp++;     /* 防止重复增加 */
     
140.                 parmnum++;  /* 参数增加一个 */
     
141.             }
     
142.         }
     
143. 
     
144.         strtemp++;
     
145.     }
     
146. 
     
147.     if (parmnum == 1)       /* 只有1个参数. */
     
148.     {
     
149.         fpname[fplcnt] = '\0';  /* 加入结束符 */
     
150. 
     
151.         if (usmart_strcmp(fpname, "void") == 0)parmnum = 0; /* 参数为void,表示没有参数. */
     
152.     }
     
153. 
     
154.     *pnum = parmnum;/* 记录参数个数 */
     
155.     *fname = '\0';  /* 加入结束符 */
     
156.     return res;     /* 返回执行结果 */
     
157. }

复制代码

LcwSwust

虽然没用过原子的usmart，但楼主说的有道理，看来是发现了BUG。
可以试一下这些情况，比如：
fun("1,2",3)
fun("1,\"2",3)
fun(',',3)