尝试使用双指针复制字符串时出现分段错误
Segmentation fault when trying to copy string with double pointer
刚开始学习指针,我坚持使用这个输出分段错误的程序。
它应该将字符串的前 10 个字符复制到双指针指向的位置
使用 gdb 我发现 **pt=*s;产生段错误
#include <stdio.h>
#include <stdlib.h>
void str1(char *s, char **pt);
void str1(char *s, char **pt){
for(int i=0;i<10;i++){
**pt=*s;
pt++;
s++;
}
}
int main (void) {
char str[30] = "223This is test";
char *ptr;
str1(str, &ptr);
printf("%s", ptr);
return 0;
}
你可能想要这个:
#include <stdio.h>
#include <stdlib.h>
void str1(char* s, char** pt) {
char *p = malloc(100); // allocate memory for destination
*pt = p; // store it for the caller
for (int i = 0; i < 10; i++) {
*p = *s;
p++;
s++;
}
*p = 0; // put string terminator, otherwise printf won't work correctly
}
int main(void) {
char str[30] = "223This is test";
char *ptr; // for now p points nowhere
str1(str, &ptr); // now p points to the memory allocated in str1
printf("%s", ptr);
free(ptr); // free memory for completeness
return 0;
}
首先ptr
没有被初始化,你不能真正使用它,除非你为它预留space或者在其中存储一个有效的内存地址,即让它指向一些有效的变量。
char *ptr = malloc(11);
然后你需要在函数中适当地增加它:
(*pt)++;
复制完成后,您需要以 null 终止 char 数组,以便将其视为字符串,也就是以 null 终止的 char 数组。
**pt = '[=12=]';
现在,由于 ptr
作为指向指针的指针传递,调用者知道增量,在本例中为 main
,因此当您尝试打印它时,它什么也不打印,因为它是指向char数组的末尾,我们需要把它带回开头。
*pt -= 10;
更正了以您的注释为基础的代码:
#include <stdio.h>
#include <stdlib.h>
#define SIZE 10
void str1(char *s, char **pt) {
for (int i = 0; i < SIZE; i++) {
**pt = *s;
(*pt)++; //properly increment pt
s++;
}
**pt = '[=14=]'; //null terminate copied string
//since ptr was passed as **, the increment is known by the caller
//now ptr will be pointing to the end of the string
//we have to bring it back to the beginning
*pt -= SIZE;
}
int main(void) {
char str[] = "223This is test";
char *ptr = malloc(SIZE + 1); //space for 10 character + null-terminator
//check for allocation errors
if(ptr == NULL){
perror("malloc");
return EXIT_FAILURE;
}
str1(str, &ptr);
printf("%s", ptr);
free(ptr);
return EXIT_SUCCESS;
}
刚开始学习指针,我坚持使用这个输出分段错误的程序。 它应该将字符串的前 10 个字符复制到双指针指向的位置 使用 gdb 我发现 **pt=*s;产生段错误
#include <stdio.h>
#include <stdlib.h>
void str1(char *s, char **pt);
void str1(char *s, char **pt){
for(int i=0;i<10;i++){
**pt=*s;
pt++;
s++;
}
}
int main (void) {
char str[30] = "223This is test";
char *ptr;
str1(str, &ptr);
printf("%s", ptr);
return 0;
}
你可能想要这个:
#include <stdio.h>
#include <stdlib.h>
void str1(char* s, char** pt) {
char *p = malloc(100); // allocate memory for destination
*pt = p; // store it for the caller
for (int i = 0; i < 10; i++) {
*p = *s;
p++;
s++;
}
*p = 0; // put string terminator, otherwise printf won't work correctly
}
int main(void) {
char str[30] = "223This is test";
char *ptr; // for now p points nowhere
str1(str, &ptr); // now p points to the memory allocated in str1
printf("%s", ptr);
free(ptr); // free memory for completeness
return 0;
}
首先ptr
没有被初始化,你不能真正使用它,除非你为它预留space或者在其中存储一个有效的内存地址,即让它指向一些有效的变量。
char *ptr = malloc(11);
然后你需要在函数中适当地增加它:
(*pt)++;
复制完成后,您需要以 null 终止 char 数组,以便将其视为字符串,也就是以 null 终止的 char 数组。
**pt = '[=12=]';
现在,由于 ptr
作为指向指针的指针传递,调用者知道增量,在本例中为 main
,因此当您尝试打印它时,它什么也不打印,因为它是指向char数组的末尾,我们需要把它带回开头。
*pt -= 10;
更正了以您的注释为基础的代码:
#include <stdio.h>
#include <stdlib.h>
#define SIZE 10
void str1(char *s, char **pt) {
for (int i = 0; i < SIZE; i++) {
**pt = *s;
(*pt)++; //properly increment pt
s++;
}
**pt = '[=14=]'; //null terminate copied string
//since ptr was passed as **, the increment is known by the caller
//now ptr will be pointing to the end of the string
//we have to bring it back to the beginning
*pt -= SIZE;
}
int main(void) {
char str[] = "223This is test";
char *ptr = malloc(SIZE + 1); //space for 10 character + null-terminator
//check for allocation errors
if(ptr == NULL){
perror("malloc");
return EXIT_FAILURE;
}
str1(str, &ptr);
printf("%s", ptr);
free(ptr);
return EXIT_SUCCESS;
}