內存資源（的strdup）

-2

我打電話的strdup複製在set_device（DEVNAME） set_device（DEVNAME） '卡' 的字符串，然後我用 '卡'，打開攪拌機：內存資源（的strdup）

devname是在格式HW： 0 /麥克風

static char *card, *channel; 
static snd_mixer_t *handle = NULL; 
static snd_mixer_elem_t *elem = NULL; 
static long min, max, vol; 

static void open_mixer(void) 
{ 
     int err; 
     static snd_mixer_selem_id_t *sid = NULL; 
     if ((err = snd_mixer_open (&handle, 0)) < 0) { 
      return; 
     } 
     if ((err = snd_mixer_attach (handle, card)) < 0) { /* memory leak */ 
      goto error; 
     } 
     if ((err = snd_mixer_selem_register (handle, NULL, NULL)) < 0) { 
      goto error; 
     } 
     if ((err = snd_mixer_load (handle)) < 0) { 
      goto error; 
     } 
     snd_mixer_selem_id_malloc(&sid); 
     if (sid == NULL) 
      goto error; 
     snd_mixer_selem_id_set_name(sid, channel); 
     if (!(elem = snd_mixer_find_selem(handle, sid))) { 
      goto error; 
     } 
     if (!snd_mixer_selem_has_playback_volume(elem)) { 
      goto error; 
     } 
     snd_mixer_selem_get_playback_volume_range(elem, &min, &max); 
     if ((max - min) <= 0) { 
      goto error; 
     } 
     snd_mixer_selem_id_free(sid); 
     return; 

error: 
     if (sid) 
      snd_mixer_selem_id_free(sid); 
     if (handle) { 
      snd_mixer_close(handle); 
      handle = NULL; 
     } 
     return; 
} 

int set_device(const char *devname) 
{ 
     int i; 

     if (card) free(card); 
     card = strdup(devname); 
     if(!card) return -1; 

     i = strcspn(card, "/"); 
     if(i == strlen(card)) { 
      channel = "Mic"; 
     } else { 
      card[i] = 0; 
      channel = card + i + 1; 
     } 
     open_mixer(); 
     if (!handle) { 
      fprintf(stderr, "mixer: Can't open mixer %s, volume unavailable.\n", card); 
      return -1; 
     } 
     return 0; 
}

請幫助我，以防止內存泄漏的strdup調用

來源

2013-08-26 user1840007

避免內存泄漏的關鍵是釋放分配的內存，一旦你完成它... –

static char * card = NULL，* channel = NULL;您無法將「Mic」分配給頻道，因爲它沒有任何與其關聯的內存，只是一個指針。用char * channel [10] = {}可能會更好。 – Jiminion

功能後的strdup分配內存的malloc。如果你想避免內存泄漏，因爲strdup你必須當你不再使用數據時，釋放strdup的返回值（在你的情況下變量卡）。

這裏描述的是這個的strdup人的部分：

char *strdup(const char *s);

的的strdup（）函數返回一個指向一個新的字符串，是字符串s的副本。用於新字符串的存儲器是通過malloc（3）獲得的，並且可以是通過免費獲得（3）。

來源

2013-08-26 21:38:35 Zerkan

內存資源（的strdup）

回答

相關問題