Changeset 46c366a825accfc0d1b7b7f446c820f6713037fd

Show
Ignore:
Timestamp:
19/08/07 19:11:08 (1 year ago)
Author:
Rémi Denis-Courmont <rem@videolan.org>
git-committer:
Rémi Denis-Courmont <rem@videolan.org> 1187543468 +0000
git-parent:

[397710b25f3cbbf3e2dfcf1cbe28e408f610e4a2]

git-author:
Rémi Denis-Courmont <rem@videolan.org> 1187543468 +0000
Message:

Fix a bunch of warnings / undefined pointers
By the way, using asprintf(&ptr)/free(ptr) to determine the length of a string
is really lame, when you can do the same with snprintf(NULL, 0);

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • src/input/control.c

    r662ed77 r46c366a  
    191191 
    192192            p_info = p_cat->pp_infos[i]; 
    193             vasprintf( &p_info->psz_value, psz_format, args ); 
     193            if( vasprintf( &p_info->psz_value, psz_format, args ) == -1 ) 
     194                p_info->psz_value = NULL; 
    194195 
    195196            vlc_mutex_unlock( &p_input->p->input.p_item->lock ); 
    …… 
    307308            if( !p_bkmk->psz_name ) 
    308309            { 
    309                  asprintf( &p_bkmk->psz_name, _("Bookmark %i"), 
    310                            p_input->p->i_bookmark ); 
     310                 if( asprintf( &p_bkmk->psz_name, _("Bookmark %i"), 
     311                               p_input->p->i_bookmark ) == -1 ) 
     312                     p_bkmk->psz_name = NULL; 
    311313            } 
    312314 
    …… 
    519521            } 
    520522 
    521             asprintf( &p_input->p->input.p_item->ppsz_options[i], 
    522                       "%s=%s", psz_option, psz_value ) ; 
     523            if( asprintf( &p_input->p->input.p_item->ppsz_options[i], 
     524                          "%s=%s", psz_option, psz_value ) == -1 ) 
     525                p_input->p->input.p_item->ppsz_options[i] = NULL; 
    523526            vlc_mutex_unlock( &p_input->p->input.p_item->lock ); 
    524527 
    …… 
    614617    for( i = 0; i < p_input->p->i_bookmark; i++ ) 
    615618    { 
    616         asprintf( &psz_value, "{name=%s,bytes="I64Fd",time="I64Fd"}", 
    617                   p_input->p->bookmark[i]->psz_name, 
    618                   p_input->p->bookmark[i]->i_byte_offset, 
    619                   p_input->p->bookmark[i]->i_time_offset/1000000 ); 
    620         i_len += strlen( psz_value ); 
    621         free( psz_value ); 
     619        i_len += snprintf( NULL, 0, "{name=%s,bytes="I64Fd",time="I64Fd"}", 
     620                           p_input->p->bookmark[i]->psz_name, 
     621                           p_input->p->bookmark[i]->i_byte_offset, 
     622                           p_input->p->bookmark[i]->i_time_offset/1000000 ); 
    622623    } 
    623624    for( i = 0; i < p_input->p->i_bookmark; i++ )