局部变量的地址做为函数参数传递容易错误？

zhutroy

原帖由 bruceteen 于 2008-12-11 18:13 发表
看不出任何问题
（
有个小bug，但和本问题无关
%x对应的是int类型，但你给的是指针类型，所以要么将指针强制转化为int(可能失去精度)，要么使用%p
）

会不会不安全？


以前懒惰，喜欢这样传递

zhutroy

int  read_data(u_int8 base_addr, u_int8 register_addr, u_int8 *udata) {         &nbsp;&nbsp;&nbsp;unsigned char data[64];    &nbsp;&nbsp;&nbsp;unsigned long length = 64; &nbsp;&nbsp;&nbsp;memset(data, 0,64); &nbsp;&nbsp;&nbsp;data[0] = 1;    &nbsp;&nbsp;&nbsp;data[1] = 0x80; &nbsp;&nbsp;&nbsp;data[2] = base_addr;    &nbsp;&nbsp;&nbsp;data[3] = register_addr;    &nbsp;&nbsp;&nbsp;usb_control_read(PDEVICE, data, length);   &nbsp;&nbsp;&nbsp;if(data[0]==0x01) &nbsp;&nbsp;&nbsp;&nbsp;{ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;*udata = data[1];                   &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;*udata  = 9; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf("read address= 0x%x 0x%x data = 0x%x\n ",base_addr,register_addr,  *udata);          &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf("read address= 0x%x 0x%x addr = 0x%x\n ",base_addr,register_addr,  udata); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return 1; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}         &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return 0; } void SoftReset() { &nbsp;&nbsp;&nbsp;u_int8 loop=0;         &nbsp;&nbsp;&nbsp;u_int8 state = 0xff;         &nbsp;&nbsp;&nbsp;printf("state= 0x%x\n", state)         &nbsp;&nbsp;&nbsp;read_data(0x06, 0x07, &state);         &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;printf(" state= 0x%x\n", State);         &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;for(loop=0; loop<10; loop++) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf(" stateaddr =0x%x  data= 0x%x\n",&state , state ); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if(state!=0x05) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;printf("state= 0x%x\n", state );                                 &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;read_data(0x06, 0x07, &state); &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;} &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{ &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;break; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}          &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}                }

state= 0xff
read address= 0x6 0x7 data = 0x9
read address= 0x6 0x7 addr = 0x23eae7
state= 0x0
stateaddr =0x23eae7  data= 0x0
state= 0x0

read address= 0x6 0x7 data = 0x9
read address= 0x6 0x7 addr = 0x23eae7
stateaddr =0x23eae7  data= 0x2
state= 0x2

read address= 0x6 0x7 data = 0x9
read address= 0x6 0x7 addr = 0x23eae7
stateaddr =0x23eae7  data= 0x0
state= 0x0

read address= 0x6 0x7 data = 0x9
read address= 0x6 0x7 addr = 0x23eae7
stateaddr =0x23eae7  data= 0x3
state= 0x3

read address= 0x6 0x7 data = 0x9
read address= 0x6 0x7 addr = 0x23eae7
stateaddr =0x23eae7  data= 0x3
state= 0x3

[ 本帖最后由 zhutroy 于 2008-12-11 20:18 编辑 ]

zhutroy

程序比较乱；上面的程序删除了一些无关的打印信息

所以没有对齐

zhutroy

u_int8 state = 0xff;   

定以成静态变量, 就没有问题了,

u_int8 state = 0xff;         void SoftReset() {    u_int8 loop=0;                printf("state= 0x%x\n", state)            read_data(0x06, 0x07, &state);                      printf(" state= 0x%x\n", State);                 for(loop=0; loop<10; loop++)         {           printf(" stateaddr =0x%x  data= 0x%x\n",&state , state );           if(state!=0x05)           {             printf("state= 0x%x\n", state );                                             read_data(0x06, 0x07, &state);           }           else             {                          break;             }                  }                }

[ 本帖最后由 zhutroy 于 2008-12-11 20:19 编辑 ]

meishu

楼主 unsigned char就不要用%x打印出来了。
或者unsigned long xxx = *data, 然后再打印xxx.

雨过白鹭洲

开头的那个程序肯定没有问题的

samon_fu

局部变量放在栈里面，涉及到入栈出栈的顺序，所以有人说会有危险。

zhutroy

原帖由 samon_fu 于 2008-12-11 23:02 发表
局部变量放在栈里面，涉及到入栈出栈的顺序，所以有人说会有危险。

这些程序 以前在驱动里都没有问题

编译在mplayer播放器 才出现的

比较郁闷的是，我写个小测试get(&a)加在前面; 它又正常了，去掉测试程序， 又不行了

明天需要仔细debug

samon_fu

检查你程序的逻辑。

建议你不要去怀疑是时序的问题，一般出这种问题，都是对变量的使用的问题，主要是逻辑出了问题了。

zhutroy

原帖由 samon_fu 于 2008-12-12 04:29 发表
检查你程序的逻辑。

建议你不要去怀疑是时序的问题，一般出这种问题，都是对变量的使用的问题，主要是逻辑出了问题了。

其实这个问题可以用其他方式解决， 比如静态变量， 只是以前写的一些程序用局部变量这样做过， 很怕哪天会出问题

[ 本帖最后由 zhutroy 于 2008-12-12 09:42 编辑 ]