keil编程遇到指针问题，用结构体指针编译正确但是不能在开发板运行

DChenT · 发表于 2020-8-7 10:43:40

#include "LED.h"
#include "stm32f10x.h"
void LED_init(void)
{
GPIO_InitTypeDef *GPIO_InitStructure;
RCC_APB2PeriphClockCmd(RCC_APB2Periph_GPIOB|RCC_APB2Periph_GPIOE,ENABLE);  //使能时钟 enable disable

GPIO_InitStructure->GPIO_Mode=  GPIO_Mode_Out_PP;
GPIO_InitStructure->GPIO_Pin= GPIO_Pin_5;
GPIO_InitStructure->GPIO_Speed=  GPIO_Speed_50MHz;//GPIOB_5定为50MHz推挽输出
GPIO_Init(GPIOB,GPIO_InitStructure);
GPIO_SetBits(GPIOB,GPIO_Pin_5); //PB5输出高

GPIO_InitStructure->GPIO_Mode=  GPIO_Mode_Out_PP;
GPIO_InitStructure->GPIO_Pin= GPIO_Pin_5;
GPIO_InitStructure->GPIO_Speed=  GPIO_Speed_50MHz;//GPIOB_5定为50MHz推挽输出
GPIO_Init(GPIOE,GPIO_InitStructure);
GPIO_SetBits(GPIOE,GPIO_Pin_5); //PE5输出高
}
请问 GPIO_InitTypeDef *GPIO_InitStructure（编译正确）;结构体指针为什么不行，写成GPIO_InitTypeDef GPIO_InitStructure（后面相应都改了编译都对）反而可以运行

a5820736 · 发表于 2020-8-7 10:43:41

这个问题我在野火那边也回答过，好多人都不清楚。。。
局部变量这是个重点，定义时不赋值直接用它很危险，指针就更危险了。
你定义一个指针但没有实际分配给他一个地址，所以人家是个野指针，可能指null，可能乱指，导致程序乱跑。
你应该在定义指针时malloc一个地址给他，或者之前就定义一个GPIO_InitTypeDef变量，把地址给这个指针(这样就没意义了).

叶子君 · 发表于 2020-8-7 17:25:34

本帖最后由叶子君于 2020-8-7 17:30 编辑

个人愚见，第一点，局部变量没赋值，都是默认随机数赋值，所以第一点，你弄了个局部变量指针没赋值是不安全的，也就是野指针
第二点，指针是需要分配空间才能用的。所以加上一句 GPIO_InitStructure = malloc（sizeof（GPIO_InitTypeDef））；
https://blog.csdn.net/ytuglt/article/details/46908583
如果用的局部变量的话，则空间会自动帮你申请好的了。

姚先起 · 发表于 2020-8-7 18:23:09

第一，你直接定义 *p p的值是不确定的，也是就野指针，对野指针进行赋值操作会导致内存异常。应该是GPIO_InitTypeDef *GPIO_InitStructure;GPIO_InitStructure=(GPIO_InitTypeDef )malloc(sizeof(GPIO_InitTypeDef ));然后这个变量才能正常使用，使用完必须使用free（GPIO_InitStructure）；不然该变量占用的内存将一直被使用。