sprintf引发的死循环

本文讨论了在编程中使用不安全的sprintf函数所引起的潜在问题,特别是当错误地估计内存大小时,可能导致的死循环或缓冲区溢出。通过分析错误代码,提醒开发者注意此类安全风险,并提倡使用更安全的字符串处理函数。

摘要生成于 C知道 ,由 DeepSeek-R1 满血版支持, 前往体验 >

调用了不安全的sprintf,一开始自信满满觉得反正自己知道源和目的变量的内存,复制过去不会溢出的,所以就用这个不安全的函数吧

错误代码:

int get_len(char *buffer, int len){
	int i = 0;
	int num = 0;
	for(i = 0; i < len - 1; ++i){
		printf("bei:%d\n",i);
		char temp[1] = {0};
		int dig = 0;
		if(sprintf(temp,"%x",buffer[i]) == -1){
			printf("over flow\n");
		}
		if(temp[0] >= 'a'){
			dig = temp[0] - 'a' + 10;
		}else{
			dig = temp[0] - '0';
		}
		sleep(1);
		printf("afi:%d\n",i);
		printf("num:%d %d\n",dig,num);
		num = num*16 + dig;	
	}
	return num;
}
printf("bei:%d\n",i);
printf("afi:%d\n",i);

分别打印sprinf语句执行前后的i值,打印出来结果可以观察到sprintf后i值被修改了,原因是
sprintf(temp,"%x",buffer[i])

buffer[i]虽然是一个字节但是"%x"后变为4字节,而temp就一个字节,所以产生溢出,而溢出时候又恰好修改了局部变量i,所以最终死循环

#include <string.h> #include "board.h" #include "stm32l1xx_hal.h" #include "sht1x.h" #include "tim-board.h" #include "timer_handles.h" #include <math.h> #include "hal_key.h" #include "adc_reader.h" #include "sht1x.h" #include "sht3x.h" #include "NS_Radio.h" void Init() { BoardInitMcu(); BoardInitPeriph(); keys_init(); setTimer2Callback(Time2Handler); Tim2McuInit(1); } float temp=0,hum=0,light=0; uint8_t data[30]; uint8_t led_state = 0; void HandleKeyPress(void) { uint8_t send_value = 0x05; // ????0x05 if(isKey2Pressed()) { // ????? SX1276Send(uint8_t *send_value,30); // ???????? send_value = (send_value == 0x05) ? 0x06 : 0x05; // ?????? while(isKey3Pressed()); } } int main() { Init(); OLED_Init(); hal_temHumInit(); ADCS_Init(); NS_RadioInit(433000000,10,1000,1000); OLED_Display_On(); uint8_t tx_data[2]; while(1) { HAL_Delay(1000); AdcScanChannel(); call_sht11(&temp,&hum); light=((5.0/2.0)*AdcReadCh0())*100.0; OLED_Clear(); OLED_ShowCHinese(0,0,29); OLED_ShowCHinese(16,0,30); OLED_ShowString(32,0,(uint8_t *)":"); memset(data,0,30); sprintf((char *)data,"%.2f Lx",light); OLED_ShowString(48,0,data); OLED_ShowCHinese(0,3,22); OLED_ShowCHinese(16,3,23); OLED_ShowString(32,3,(uint8_t *)":"); memset(data,0,30); sprintf((char *)data,"%.2f C",temp); OLED_ShowString(48,3,data); OLED_ShowCHinese(0,6,24); OLED_ShowCHinese(16,6,23); OLED_ShowString(32,6,(uint8_t *)":"); memset(data,0,30); sprintf((char *)data,"%.2f %%Rh",hum); OLED_ShowString(48,6,data); } }为什么显示数据类型不对
07-02
我再使用多进程时,在下列代码中unsigned WINAPI MyThreadFunction(LPVOID lpParam) { while (openFlag) { UF_UI_lock_ug_access(UF_UI_FROM_CUSTOM); //给主线程加锁(不锁上的话,在点击应用后再点击调取命令会崩掉或死循环的可能) char message[256]; //错误信息 getVariableDir("VT_APPLICATION_DIR", czApplicationDir); if (!isDirHas(czApplicationDir)) { UF_UI_open_listing_window(); UF_UI_write_listing_window("读取 VT_APPLICATION_DIR 环境变量失败 \n"); return false; } char czCfgDataP[256]; //表格完整路径 strcpy_s(czCfgDataP, czApplicationDir); strcat_s(czCfgDataP, "\\data\\标准化配置\\设计标准配置表.xls"); if (!isDirHas_Chs(czCfgDataP)) { sprintf_s(message,"(%s)不存在 \n", czCfgDataP); UF_UI_open_listing_window(); UF_UI_write_listing_window(message); return false; } Book* book; if (!load(czCfgDataP, &book)) { sprintf_s(message, "加载(%s) 失败 \n", czCfgDataP); UF_UI_open_listing_window(); UF_UI_write_listing_window(message); book->release(); return false; } Sheet* voiceSheet;//声音数据页 if (!getSheet(book, "材料", &voiceSheet)) { UF_UI_open_listing_window(); UF_UI_write_listing_window("加载 (材料) 数据 失败 \n"); book->release(); return false; } vAllVoiceSt.clear(); vVoiceStr.clear(); if (!readStr_Col(voiceSheet, 0, vVoiceStr)) { UF_UI_open_listing_window(); UF_UI_write_listing_window("加载 (材料) 材质类型 失败 \n"); book->release(); return false; } for (size_t i = 0; i < vVoiceStr.size(); i++) { vector<CString>vTempStr; if (!readStr_Col(voiceSheet, i, vTempStr)) { continue; } vAllVoiceSt.push_back(vTempStr); break; } book->release(); HANDLE g_hMutex = CreateMutex(NULL, FALSE, NULL); if (vAllVoiceSt.size() && tempStrUi && openFlag) { string excelDate = (LPCSTR)(CStringA)(vAllVoiceSt[0][0]); if (!excelDate.empty()) { vector<NXString>tempStr; tempStr.push_back((NXString)excelDate.c_str()); tempStrUi->SetEnable(true); tempStrUi->SetValue(tempStr); tempStrUi->SetEnable(false); } else { vector<NXString>tempStr; tempStr.push_back(""); tempStrUi->SetEnable(true); tempStrUi->SetValue(tempStr); tempStrUi->SetEnable(false); } } ReleaseMutex(g_hMutex); UF_UI_unlock_ug_access(UF_UI_FROM_CUSTOM); //给主线程关锁 Sleep(500); // Simulate work with a sleep call. } return 0; } tempStrUi->SetValue(tempStr);这句代码总是会导致NX2206崩溃该如何解决?
最新发布
07-24
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值