# HG changeset patch # User mru # Date 1182640232 0 # Node ID da71207a7cf1bee7894764375692fb53976f91a2 # Parent 4c2924cebe003cf51d6ba564af273b2415c37df0 use new string functions based on patch by Reimar Dffinger diff -r 4c2924cebe00 -r da71207a7cf1 avio.c --- a/avio.c Sat Jun 23 12:49:36 2007 +0000 +++ b/avio.c Sat Jun 23 23:10:32 2007 +0000 @@ -19,6 +19,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ #include "avformat.h" +#include "avstring.h" static int default_interrupt_cb(void); @@ -169,7 +170,7 @@ void url_get_filename(URLContext *h, char *buf, int buf_size) { - pstrcpy(buf, buf_size, h->filename); + av_strlcpy(buf, h->filename, buf_size); } diff -r 4c2924cebe00 -r da71207a7cf1 http.c --- a/http.c Sat Jun 23 12:49:36 2007 +0000 +++ b/http.c Sat Jun 23 23:10:32 2007 +0000 @@ -23,6 +23,7 @@ #include "network.h" #include "base64.h" +#include "avstring.h" /* XXX: POST protocol is not completely implemented because ffmpeg uses only a subset of it. */ @@ -72,7 +73,7 @@ if (port > 0) { snprintf(hoststr, sizeof(hoststr), "%s:%d", hostname, port); } else { - pstrcpy(hoststr, sizeof(hoststr), hostname); + av_strlcpy(hoststr, hostname, sizeof(hoststr)); } if (use_proxy) { diff -r 4c2924cebe00 -r da71207a7cf1 img2.c --- a/img2.c Sat Jun 23 12:49:36 2007 +0000 +++ b/img2.c Sat Jun 23 23:10:32 2007 +0000 @@ -20,6 +20,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ #include "avformat.h" +#include "avstring.h" typedef struct { int img_first; @@ -182,7 +183,7 @@ return AVERROR(ENOMEM); } - pstrcpy(s->path, sizeof(s->path), s1->filename); + av_strlcpy(s->path, s1->filename, sizeof(s->path)); s->img_number = 0; s->img_count = 0; @@ -307,7 +308,7 @@ VideoData *img = s->priv_data; img->img_number = 1; - pstrcpy(img->path, sizeof(img->path), s->filename); + av_strlcpy(img->path, s->filename, sizeof(img->path)); /* find format */ if (s->oformat->flags & AVFMT_NOFILE) diff -r 4c2924cebe00 -r da71207a7cf1 mp3.c --- a/mp3.c Sat Jun 23 12:49:36 2007 +0000 +++ b/mp3.c Sat Jun 23 23:10:32 2007 +0000 @@ -20,6 +20,7 @@ */ #include "avformat.h" #include "mpegaudio.h" +#include "avstring.h" #define ID3v2_HEADER_SIZE 10 #define ID3v1_TAG_SIZE 128 @@ -350,7 +351,7 @@ s->track = buf[126]; genre = buf[127]; if (genre <= ID3v1_GENRE_MAX) - pstrcpy(s->genre, sizeof(s->genre), id3v1_genre_str[genre]); + av_strlcpy(s->genre, id3v1_genre_str[genre], sizeof(s->genre)); return 0; } diff -r 4c2924cebe00 -r da71207a7cf1 nutdec.c --- a/nutdec.c Sat Jun 23 12:49:36 2007 +0000 +++ b/nutdec.c Sat Jun 23 23:10:32 2007 +0000 @@ -23,6 +23,7 @@ #include "tree.h" #include "nut.h" +#include "avstring.h" #undef NDEBUG #include @@ -404,13 +405,13 @@ if(chapter_id==0 && !strcmp(type, "UTF-8")){ if (!strcmp(name, "Author")) - pstrcpy(s->author , sizeof(s->author) , str_value); + av_strlcpy(s->author , str_value, sizeof(s->author)); else if(!strcmp(name, "Title")) - pstrcpy(s->title , sizeof(s->title) , str_value); + av_strlcpy(s->title , str_value, sizeof(s->title)); else if(!strcmp(name, "Copyright")) - pstrcpy(s->copyright, sizeof(s->copyright), str_value); + av_strlcpy(s->copyright, str_value, sizeof(s->copyright)); else if(!strcmp(name, "Description")) - pstrcpy(s->comment , sizeof(s->comment) , str_value); + av_strlcpy(s->comment , str_value, sizeof(s->comment)); } } diff -r 4c2924cebe00 -r da71207a7cf1 rmdec.c --- a/rmdec.c Sat Jun 23 12:49:36 2007 +0000 +++ b/rmdec.c Sat Jun 23 23:10:32 2007 +0000 @@ -20,7 +20,7 @@ */ #include "avformat.h" #include "rm.h" - +#include "avstring.h" static void get_str(ByteIOContext *pb, char *buf, int buf_size) { @@ -175,8 +175,7 @@ } } else { st->codec->codec_id = CODEC_ID_NONE; - pstrcpy(st->codec->codec_name, sizeof(st->codec->codec_name), - buf); + av_strlcpy(st->codec->codec_name, buf, sizeof(st->codec->codec_name)); } if (read_all) { get_byte(pb); diff -r 4c2924cebe00 -r da71207a7cf1 rtsp.c --- a/rtsp.c Sat Jun 23 12:49:36 2007 +0000 +++ b/rtsp.c Sat Jun 23 23:10:32 2007 +0000 @@ -23,6 +23,7 @@ #include #include /* for select() prototype */ #include "network.h" +#include "avstring.h" #include "rtp_internal.h" @@ -405,11 +406,11 @@ } break; case 's': - pstrcpy(s->title, sizeof(s->title), p); + av_strlcpy(s->title, p, sizeof(s->title)); break; case 'i': if (s->nb_streams == 0) { - pstrcpy(s->comment, sizeof(s->comment), p); + av_strlcpy(s->comment, p, sizeof(s->comment)); break; } break; @@ -456,7 +457,7 @@ } } /* put a default control url */ - pstrcpy(rtsp_st->control_url, sizeof(rtsp_st->control_url), s->filename); + av_strlcpy(rtsp_st->control_url, s->filename, sizeof(rtsp_st->control_url)); break; case 'a': if (strstart(p, "control:", &p) && s->nb_streams > 0) { @@ -469,10 +470,10 @@ url_split(proto, sizeof(proto), NULL, 0, NULL, 0, NULL, NULL, 0, p); if (proto[0] == '\0') { /* relative control URL */ - pstrcat(rtsp_st->control_url, sizeof(rtsp_st->control_url), "/"); - pstrcat(rtsp_st->control_url, sizeof(rtsp_st->control_url), p); + av_strlcat(rtsp_st->control_url, "/", sizeof(rtsp_st->control_url)); + av_strlcat(rtsp_st->control_url, p, sizeof(rtsp_st->control_url)); } else { - pstrcpy(rtsp_st->control_url, sizeof(rtsp_st->control_url), p); + av_strlcpy(rtsp_st->control_url, p, sizeof(rtsp_st->control_url)); } } else if (strstart(p, "rtpmap:", &p)) { /* NOTE: rtpmap is only supported AFTER the 'm=' tag */ @@ -749,14 +750,14 @@ memset(reply, 0, sizeof(RTSPHeader)); rt->seq++; - pstrcpy(buf, sizeof(buf), cmd); + av_strlcpy(buf, cmd, sizeof(buf)); snprintf(buf1, sizeof(buf1), "CSeq: %d\r\n", rt->seq); - pstrcat(buf, sizeof(buf), buf1); + av_strlcat(buf, buf1, sizeof(buf)); if (rt->session_id[0] != '\0' && !strstr(cmd, "\nIf-Match:")) { snprintf(buf1, sizeof(buf1), "Session: %s\r\n", rt->session_id); - pstrcat(buf, sizeof(buf), buf1); + av_strlcat(buf, buf1, sizeof(buf)); } - pstrcat(buf, sizeof(buf), "\r\n"); + av_strlcat(buf, "\r\n", sizeof(buf)); #ifdef DEBUG printf("Sending:\n%s--\n", buf); #endif @@ -795,14 +796,14 @@ reply->status_code = atoi(buf1); } else { rtsp_parse_line(reply, p); - pstrcat(rt->last_reply, sizeof(rt->last_reply), p); - pstrcat(rt->last_reply, sizeof(rt->last_reply), "\n"); + av_strlcat(rt->last_reply, p, sizeof(rt->last_reply)); + av_strlcat(rt->last_reply, "\n", sizeof(rt->last_reply)); } line_count++; } if (rt->session_id[0] == '\0' && reply->session_id[0] != '\0') - pstrcpy(rt->session_id, sizeof(rt->session_id), reply->session_id); + av_strlcpy(rt->session_id, reply->session_id, sizeof(rt->session_id)); content_length = reply->content_length; if (content_length > 0) { @@ -947,7 +948,7 @@ rtp_opened: port = rtp_get_local_port(rtsp_st->rtp_handle); if (transport[0] != '\0') - pstrcat(transport, sizeof(transport), ","); + av_strlcat(transport, ",", sizeof(transport)); snprintf(transport + strlen(transport), sizeof(transport) - strlen(transport) - 1, "RTP/AVP/UDP;unicast;client_port=%d-%d", port, port + 1); @@ -956,14 +957,14 @@ /* RTP/TCP */ else if (protocol_mask & (1 << RTSP_PROTOCOL_RTP_TCP)) { if (transport[0] != '\0') - pstrcat(transport, sizeof(transport), ","); + av_strlcat(transport, ",", sizeof(transport)); snprintf(transport + strlen(transport), sizeof(transport) - strlen(transport) - 1, "RTP/AVP/TCP"); } else if (protocol_mask & (1 << RTSP_PROTOCOL_RTP_UDP_MULTICAST)) { if (transport[0] != '\0') - pstrcat(transport, sizeof(transport), ","); + av_strlcat(transport, ",", sizeof(transport)); snprintf(transport + strlen(transport), sizeof(transport) - strlen(transport) - 1, "RTP/AVP/UDP;multicast"); diff -r 4c2924cebe00 -r da71207a7cf1 utils.c --- a/utils.c Sat Jun 23 12:49:36 2007 +0000 +++ b/utils.c Sat Jun 23 23:10:32 2007 +0000 @@ -21,6 +21,7 @@ #include "avformat.h" #include "allformats.h" #include "opt.h" +#include "avstring.h" #undef NDEBUG #include @@ -362,7 +363,7 @@ ic->pb = *pb; ic->duration = AV_NOPTS_VALUE; ic->start_time = AV_NOPTS_VALUE; - pstrcpy(ic->filename, sizeof(ic->filename), filename); + av_strlcpy(ic->filename, filename, sizeof(ic->filename)); /* allocate private data */ if (fmt->priv_data_size > 0) { @@ -2840,7 +2841,7 @@ } if (port_ptr) *port_ptr = port; - pstrcpy(path, path_size, p); + av_strlcpy(path, p, path_size); } void av_set_pts_info(AVStream *s, int pts_wrap_bits,