修改指針值

Question

首先，我已經用C

unsigned char * text ="Test program"; 
int _size = strlen(text); 
unsigned char * str = malloc(sizeof(text)); 
memcpy(str, text, _size); 
printf("Before(%d): %s\n", _size, str); 
for(i=0;i < _size; i++) { 
    str[i] -= 13; //rot13 
} 
printf("After: (%d): %s\n", strlen(str), str); 

創建一個簡單的程序，它運行正常。然而，當我提出這個代碼的Linux內核，它似乎無法工作

unsigned char * str; 
len = min(count, log->size - read->off); 

/* Allocate the memory for storing plain text */ 
str = kmalloc(len, GFP_KERNEL); 
if(str == NULL) { 
    printk(KERN_ERR "logger: failed to allocate buffer\n"); 
    return -ENOMEM; 
} 
memcpy(str, log->buf + read->off, len); 
/* Start: Add a simple rot13 encryption here */ 
for(i=0;i < strlen(str); i++) 
    str[i] -= 13; //rot13 
/* End: Add a simple rot13 encryption here */ 

if (copy_to_user(buf, str, len)) 
    return -EFAULT; 

if(str != NULL) { 
    kfree(str); 
} 

的問題來自於因爲如果它刪除，程序運行的情況下，原來下面的代碼

for(i=0;i < strlen(str); i++) 
    str[i] -= 13; //rot13 

。我在這裏錯過了什麼嗎？

Answer 1

您還沒有終止str與，所以你很可能只是運行在緩衝區的末尾和跺腳內存。

變化：

str = kmalloc(len, GFP_KERNEL); 

到：

str = kmalloc(len + 1, GFP_KERNEL); // allocate additional char for terminator 

和變化：

memcpy(str, log->buf + read->off, len); 

到：

memcpy(str, log->buf + read->off, len); 
str[len] = '\0';     // put terminator at end of string 

Answer 2

的問題：sizeof(text)返回指針的大小，而不是字符串的長度text指向。還要記住，所有的字符串都有一個額外的字符來終止字符串。這一切都意味着你寫入和讀取超出了你分配的內存，這是未定義的行爲，這意味着任何事情都可能發生。

此外，文字字​​符串實際上是不變的（const char *）。

最後，您可能想要了解ROT13，因爲您正在做的是而不是 ROT13加密。

Answer 3

如果您正在處理字符串，請嘗試使用strncpy（）而不是memcpy。 'coz會自動結束一個NULL字符，並且您可以安全地從緩衝區運行。但在這種情況下，我不太確定問題到底是什麼，除非您提供有關此問題的更多詳細信息。

對於任何內核編程錯誤，關鍵是要添加調試/打印和收集與發生的事情相同的數據。如果那不能幫助你調試自己，那將有助於他人更好地幫助你。