X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Ftext%2Fstrings.c;h=cd110e73325ce6b56266b2d573c2818067c5e2a2;hb=9f1a2dccb55a21de912f580e95100f5a99ecbd06;hp=fac02508259fae6fdf7d6c783a611e856f3cb5cd;hpb=b149f96f6466e1b94c773db3472121dfdfafbbb3;p=vlc diff --git a/src/text/strings.c b/src/text/strings.c index fac0250825..cd110e7332 100644 --- a/src/text/strings.c +++ b/src/text/strings.c @@ -2,6 +2,7 @@ * strings.c: String related functions ***************************************************************************** * Copyright (C) 2006 the VideoLAN team + * Copyright (C) 2008-2009 Rémi Denis-Courmont * $Id$ * * Authors: Antoine Cellerier @@ -35,6 +36,7 @@ /* Needed by str_format_time */ #include +#include /* Needed by str_format_meta */ #include @@ -47,90 +49,7 @@ #include /** - * Unescape URI encoded string - * \return decoded duplicated string - */ -char *unescape_URI_duplicate( const char *psz ) -{ - char *psz_dup = strdup( psz ); - unescape_URI( psz_dup ); - return psz_dup; -} - -/** - * Unescape URI encoded string in place - * \return nothing - */ -void unescape_URI( char *psz ) -{ - unsigned char *in = (unsigned char *)psz, *out = in, c; - if( psz == NULL ) - return; - - while( ( c = *in++ ) != '\0' ) - { - switch( c ) - { - case '%': - { - char val[5], *pval = val; - unsigned long cp; - - switch( c = *in++ ) - { - case '\0': - return; - - case 'u': - case 'U': - if( ( *pval++ = *in++ ) == '\0' ) - return; - if( ( *pval++ = *in++ ) == '\0' ) - return; - c = *in++; - - default: - *pval++ = c; - if( ( *pval++ = *in++ ) == '\0' ) - return; - *pval = '\0'; - } - - cp = strtoul( val, NULL, 0x10 ); - if( cp < 0x80 ) - *out++ = cp; - else - if( cp < 0x800 ) - { - *out++ = (( cp >> 6) | 0xc0); - *out++ = (( cp & 0x3f) | 0x80); - } - else - { - assert( cp < 0x10000 ); - *out++ = (( cp >> 12) | 0xe0); - *out++ = (((cp >> 6) & 0x3f) | 0x80); - *out++ = (( cp & 0x3f) | 0x80); - } - break; - } - - /* + is not a special case - it means plus, not space. */ - - default: - /* Inserting non-ASCII or non-printable characters is unsafe, - * and no sane browser will send these unencoded */ - if( ( c < 32 ) || ( c > 127 ) ) - *out++ = '?'; - else - *out++ = c; - } - } - *out = '\0'; -} - -/** - * Decode encoded URI string + * Decode encoded URI component. See also decode_URI(). * \return decoded duplicated string */ char *decode_URI_duplicate( const char *psz ) @@ -141,14 +60,23 @@ char *decode_URI_duplicate( const char *psz ) } /** - * Decode encoded URI string in place - * \return nothing + * Decode an encoded URI component in place. + * This function does NOT decode entire URIs. + * It decodes components (e.g. host name, directory, file name). + * Decoded URIs do not exist in the real world (see RFC3986 §2.4). + * Complete URIs are always "encoded" (or they are syntaxically invalid). + * + * Note that URI encoding is different from Javascript escaping. Especially, + * white spaces and Unicode non-ASCII code points are encoded differently. + * + * \return psz on success, NULL if it was not properly encoded */ -void decode_URI( char *psz ) +char *decode_URI( char *psz ) { unsigned char *in = (unsigned char *)psz, *out = in, c; + if( psz == NULL ) - return; + return NULL; while( ( c = *in++ ) != '\0' ) { @@ -160,14 +88,14 @@ void decode_URI( char *psz ) if( ( ( hex[0] = *in++ ) == 0 ) || ( ( hex[1] = *in++ ) == 0 ) ) - return; + return NULL; hex[2] = '\0'; *out++ = (unsigned char)strtoul( hex, NULL, 0x10 ); break; } - case '+': + case '+': /* This is HTTP forms, not URI decoding... */ *out++ = ' '; break; @@ -181,189 +109,202 @@ void decode_URI( char *psz ) } } *out = '\0'; - EnsureUTF8( psz ); + return psz; } -static inline int isurlsafe( int c ) +static inline bool isurisafe( int c ) { + /* These are the _unreserved_ URI characters (RFC3986 §2.3) */ return ( (unsigned char)( c - 'a' ) < 26 ) || ( (unsigned char)( c - 'A' ) < 26 ) || ( (unsigned char)( c - '0' ) < 10 ) - /* Hmm, we should not encode character that are allowed in URLs - * (even if they are not URL-safe), nor URL-safe characters. - * We still encode some of them because of Microsoft's crap browser. - */ - || ( strchr( "-_.", c ) != NULL ); -} - -static inline char url_hexchar( int c ) -{ - return ( c < 10 ) ? c + '0' : c + 'A' - 10; + || ( strchr( "-._~", c ) != NULL ); } -/** - * encode_URI_component - * Encodes an URI component. - * - * @param psz_url nul-terminated UTF-8 representation of the component. - * Obviously, you can't pass an URI containing a nul character, but you don't - * want to do that, do you? - * - * @return encoded string (must be free()'d) - */ -char *encode_URI_component( const char *psz_url ) +static char *encode_URI_bytes (const char *psz_uri, size_t len) { - char psz_enc[3 * strlen( psz_url ) + 1], *out = psz_enc; - const uint8_t *in; + char *psz_enc = malloc (3 * len + 1), *out = psz_enc; + if (psz_enc == NULL) + return NULL; - for( in = (const uint8_t *)psz_url; *in; in++ ) + for (size_t i = 0; i < len; i++) { - uint8_t c = *in; + static const char hex[16] = "0123456789ABCDEF"; + uint8_t c = *psz_uri; - if( isurlsafe( c ) ) - *out++ = (char)c; - else - if ( c == ' ') - *out++ = '+'; + if( isurisafe( c ) ) + *out++ = c; + /* This is URI encoding, not HTTP forms: + * Space is encoded as '%20', not '+'. */ else { *out++ = '%'; - *out++ = url_hexchar( c >> 4 ); - *out++ = url_hexchar( c & 0xf ); + *out++ = hex[c >> 4]; + *out++ = hex[c & 0xf]; } + psz_uri++; } *out++ = '\0'; - return strdup( psz_enc ); + out = realloc (psz_enc, out - psz_enc); + return out ? out : psz_enc; /* realloc() can fail (safe) */ +} + +/** + * Encodes an URI component (RFC3986 §2). + * + * @param psz_uri nul-terminated UTF-8 representation of the component. + * Obviously, you can't pass an URI containing a nul character, but you don't + * want to do that, do you? + * + * @return encoded string (must be free()'d), or NULL for ENOMEM. + */ +char *encode_URI_component( const char *psz_uri ) +{ + return encode_URI_bytes (psz_uri, strlen (psz_uri)); } + static const struct xml_entity_s { char psz_entity[8]; - uint8_t i_length; char psz_char[4]; -} p_xml_entities[] = { +} xml_entities[] = { /* Important: this list has to be in alphabetical order (psz_entity-wise) */ - { "AElig;", 6, "Æ" }, - { "Aacute;", 7, "Á" }, - { "Acirc;", 6, "Â" }, - { "Agrave;", 7, "À" }, - { "Aring;", 6, "Å" }, - { "Atilde;", 7, "Ã" }, - { "Auml;", 5, "Ä" }, - { "Ccedil;", 7, "Ç" }, - { "Dagger;", 7, "‡" }, - { "ETH;", 4, "Ð" }, - { "Eacute;", 7, "É" }, - { "Ecirc;", 6, "Ê" }, - { "Egrave;", 7, "È" }, - { "Euml;", 5, "Ë" }, - { "Iacute;", 7, "Í" }, - { "Icirc;", 6, "Î" }, - { "Igrave;", 7, "Ì" }, - { "Iuml;", 5, "Ï" }, - { "Ntilde;", 7, "Ñ" }, - { "OElig;", 6, "Œ" }, - { "Oacute;", 7, "Ó" }, - { "Ocirc;", 6, "Ô" }, - { "Ograve;", 7, "Ò" }, - { "Oslash;", 7, "Ø" }, - { "Otilde;", 7, "Õ" }, - { "Ouml;", 5, "Ö" }, - { "Scaron;", 7, "Š" }, - { "THORN;", 6, "Þ" }, - { "Uacute;", 7, "Ú" }, - { "Ucirc;", 6, "Û" }, - { "Ugrave;", 7, "Ù" }, - { "Uuml;", 5, "Ü" }, - { "Yacute;", 7, "Ý" }, - { "Yuml;", 5, "Ÿ" }, - { "aacute;", 7, "á" }, - { "acirc;", 6, "â" }, - { "acute;", 6, "´" }, - { "aelig;", 6, "æ" }, - { "agrave;", 7, "à" }, - { "aring;", 6, "å" }, - { "atilde;", 7, "ã" }, - { "auml;", 5, "ä" }, - { "bdquo;", 6, "„" }, - { "brvbar;", 7, "¦" }, - { "ccedil;", 7, "ç" }, - { "cedil;", 6, "¸" }, - { "cent;", 5, "¢" }, - { "circ;", 5, "ˆ" }, - { "copy;", 5, "©" }, - { "curren;", 7, "¤" }, - { "dagger;", 7, "†" }, - { "deg;", 4, "°" }, - { "divide;", 7, "÷" }, - { "eacute;", 7, "é" }, - { "ecirc;", 6, "ê" }, - { "egrave;", 7, "è" }, - { "eth;", 4, "ð" }, - { "euml;", 5, "ë" }, - { "euro;", 5, "€" }, - { "frac12;", 7, "½" }, - { "frac14;", 7, "¼" }, - { "frac34;", 7, "¾" }, - { "hellip;", 7, "…" }, - { "iacute;", 7, "í" }, - { "icirc;", 6, "î" }, - { "iexcl;", 6, "¡" }, - { "igrave;", 7, "ì" }, - { "iquest;", 7, "¿" }, - { "iuml;", 5, "ï" }, - { "laquo;", 6, "«" }, - { "ldquo;", 6, "“" }, - { "lsaquo;", 7, "‹" }, - { "lsquo;", 6, "‘" }, - { "macr;", 5, "¯" }, - { "mdash;", 6, "—" }, - { "micro;", 6, "µ" }, - { "middot;", 7, "·" }, - { "ndash;", 6, "–" }, - { "not;", 4, "¬" }, - { "ntilde;", 7, "ñ" }, - { "oacute;", 7, "ó" }, - { "ocirc;", 6, "ô" }, - { "oelig;", 6, "œ" }, - { "ograve;", 7, "ò" }, - { "ordf;", 5, "ª" }, - { "ordm;", 5, "º" }, - { "oslash;", 7, "ø" }, - { "otilde;", 7, "õ" }, - { "ouml;", 5, "ö" }, - { "para;", 5, "¶" }, - { "permil;", 7, "‰" }, - { "plusmn;", 7, "±" }, - { "pound;", 6, "£" }, - { "raquo;", 6, "»" }, - { "rdquo;", 6, "”" }, - { "reg;", 4, "®" }, - { "rsaquo;", 7, "›" }, - { "rsquo;", 6, "’" }, - { "sbquo;", 6, "‚" }, - { "scaron;", 7, "š" }, - { "sect;", 5, "§" }, - { "shy;", 4, "­" }, - { "sup1;", 5, "¹" }, - { "sup2;", 5, "²" }, - { "sup3;", 5, "³" }, - { "szlig;", 6, "ß" }, - { "thorn;", 6, "þ" }, - { "tilde;", 6, "˜" }, - { "times;", 6, "×" }, - { "trade;", 6, "™" }, - { "uacute;", 7, "ú" }, - { "ucirc;", 6, "û" }, - { "ugrave;", 7, "ù" }, - { "uml;", 4, "¨" }, - { "uuml;", 5, "ü" }, - { "yacute;", 7, "ý" }, - { "yen;", 4, "¥" }, - { "yuml;", 5, "ÿ" }, + { "AElig;", "Æ" }, + { "Aacute;", "Á" }, + { "Acirc;", "Â" }, + { "Agrave;", "À" }, + { "Aring;", "Å" }, + { "Atilde;", "Ã" }, + { "Auml;", "Ä" }, + { "Ccedil;", "Ç" }, + { "Dagger;", "‡" }, + { "ETH;", "Ð" }, + { "Eacute;", "É" }, + { "Ecirc;", "Ê" }, + { "Egrave;", "È" }, + { "Euml;", "Ë" }, + { "Iacute;", "Í" }, + { "Icirc;", "Î" }, + { "Igrave;", "Ì" }, + { "Iuml;", "Ï" }, + { "Ntilde;", "Ñ" }, + { "OElig;", "Œ" }, + { "Oacute;", "Ó" }, + { "Ocirc;", "Ô" }, + { "Ograve;", "Ò" }, + { "Oslash;", "Ø" }, + { "Otilde;", "Õ" }, + { "Ouml;", "Ö" }, + { "Scaron;", "Š" }, + { "THORN;", "Þ" }, + { "Uacute;", "Ú" }, + { "Ucirc;", "Û" }, + { "Ugrave;", "Ù" }, + { "Uuml;", "Ü" }, + { "Yacute;", "Ý" }, + { "Yuml;", "Ÿ" }, + { "aacute;", "á" }, + { "acirc;", "â" }, + { "acute;", "´" }, + { "aelig;", "æ" }, + { "agrave;", "à" }, + { "amp;", "&" }, + { "apos;", "'" }, + { "aring;", "å" }, + { "atilde;", "ã" }, + { "auml;", "ä" }, + { "bdquo;", "„" }, + { "brvbar;", "¦" }, + { "ccedil;", "ç" }, + { "cedil;", "¸" }, + { "cent;", "¢" }, + { "circ;", "ˆ" }, + { "copy;", "©" }, + { "curren;", "¤" }, + { "dagger;", "†" }, + { "deg;", "°" }, + { "divide;", "÷" }, + { "eacute;", "é" }, + { "ecirc;", "ê" }, + { "egrave;", "è" }, + { "eth;", "ð" }, + { "euml;", "ë" }, + { "euro;", "€" }, + { "frac12;", "½" }, + { "frac14;", "¼" }, + { "frac34;", "¾" }, + { "gt;", ">" }, + { "hellip;", "…" }, + { "iacute;", "í" }, + { "icirc;", "î" }, + { "iexcl;", "¡" }, + { "igrave;", "ì" }, + { "iquest;", "¿" }, + { "iuml;", "ï" }, + { "laquo;", "«" }, + { "ldquo;", "“" }, + { "lsaquo;", "‹" }, + { "lsquo;", "‘" }, + { "lt;", "<" }, + { "macr;", "¯" }, + { "mdash;", "—" }, + { "micro;", "µ" }, + { "middot;", "·" }, + { "nbsp;", "\xc2\xa0" }, + { "ndash;", "–" }, + { "not;", "¬" }, + { "ntilde;", "ñ" }, + { "oacute;", "ó" }, + { "ocirc;", "ô" }, + { "oelig;", "œ" }, + { "ograve;", "ò" }, + { "ordf;", "ª" }, + { "ordm;", "º" }, + { "oslash;", "ø" }, + { "otilde;", "õ" }, + { "ouml;", "ö" }, + { "para;", "¶" }, + { "permil;", "‰" }, + { "plusmn;", "±" }, + { "pound;", "£" }, + { "quot;", "\"" }, + { "raquo;", "»" }, + { "rdquo;", "”" }, + { "reg;", "®" }, + { "rsaquo;", "›" }, + { "rsquo;", "’" }, + { "sbquo;", "‚" }, + { "scaron;", "š" }, + { "sect;", "§" }, + { "shy;", "­" }, + { "sup1;", "¹" }, + { "sup2;", "²" }, + { "sup3;", "³" }, + { "szlig;", "ß" }, + { "thorn;", "þ" }, + { "tilde;", "˜" }, + { "times;", "×" }, + { "trade;", "™" }, + { "uacute;", "ú" }, + { "ucirc;", "û" }, + { "ugrave;", "ù" }, + { "uml;", "¨" }, + { "uuml;", "ü" }, + { "yacute;", "ý" }, + { "yen;", "¥" }, + { "yuml;", "ÿ" }, }; +static int cmp_entity (const void *key, const void *elem) +{ + const struct xml_entity_s *ent = elem; + const char *name = key; + + return strncmp (name, ent->psz_entity, strlen (ent->psz_entity)); +} + /** * Converts "<", ">" and "&" to "<", ">" and "&" * \param string to convert @@ -376,35 +317,42 @@ void resolve_xml_special_chars( char *psz_value ) { if( *psz_value == '&' ) { - char *psz_value1 = psz_value + 1; -#define TRY_CHAR( src, len, dst ) \ - if( !strncmp( psz_value1, src, len ) ) \ - { \ - *p_pos = dst; \ - psz_value += len + 1; \ - } - TRY_CHAR( "lt;", 3, '<' ) - else TRY_CHAR( "amp;", 4, '&' ) - else TRY_CHAR( "apos;", 5, '\'' ) - else TRY_CHAR( "gt;", 3, '>' ) - else TRY_CHAR( "quot;", 5, '"' ) -#undef TRY_CHAR - else if( *psz_value1 == '#' ) - { + if( psz_value[1] == '#' ) + { /* &#xxx; Unicode code point */ char *psz_end; - int i = strtol( psz_value+2, &psz_end, 10 ); + unsigned long cp = strtoul( psz_value+2, &psz_end, 10 ); if( *psz_end == ';' ) { - if( i >= 32 && i <= 126 ) + psz_value = psz_end + 1; + if( cp == 0 ) + (void)0; /* skip nuls */ + else + if( cp <= 0x7F ) + { + *p_pos = cp; + } + else + /* Unicode code point outside ASCII. + * &#xxx; representation is longer than UTF-8 :) */ + if( cp <= 0x7FF ) { - *p_pos = (char)i; - psz_value = psz_end+1; + *p_pos++ = 0xC0 | (cp >> 6); + *p_pos = 0x80 | (cp & 0x3F); } else + if( cp <= 0xFFFF ) { - /* Unhandled code, FIXME */ - *p_pos = *psz_value; - psz_value++; + *p_pos++ = 0xE0 | (cp >> 12); + *p_pos++ = 0x80 | ((cp >> 6) & 0x3F); + *p_pos = 0x80 | (cp & 0x3F); + } + else + if( cp <= 0x1FFFFF ) /* Outside the BMP */ + { /* Unicode stops at 10FFFF, but who cares? */ + *p_pos++ = 0xF0 | (cp >> 18); + *p_pos++ = 0x80 | ((cp >> 12) & 0x3F); + *p_pos++ = 0x80 | ((cp >> 6) & 0x3F); + *p_pos = 0x80 | (cp & 0x3F); } } else @@ -415,37 +363,21 @@ void resolve_xml_special_chars( char *psz_value ) } } else - { - const size_t i_entities = sizeof( p_xml_entities ) / - sizeof( p_xml_entities[0] ); - assert( i_entities < 128 ); - size_t step = 128>>1; - size_t i = step-1; - int cmp = -1; - while( step ) + { /* Well-known XML entity */ + const struct xml_entity_s *ent; + + ent = bsearch (psz_value + 1, xml_entities, + sizeof (xml_entities) / sizeof (*ent), + sizeof (*ent), cmp_entity); + if (ent != NULL) { - step >>= 1; - if( i >= i_entities ) - cmp = -1; - else - cmp = strncmp( psz_value1, /* Skip the & */ - p_xml_entities[i].psz_entity, - p_xml_entities[i].i_length ); - if( cmp == 0 ) - { - size_t i_len = strlen( p_xml_entities[i].psz_char ); - strncpy( p_pos, p_xml_entities[i].psz_char, i_len ); - p_pos += i_len - 1; - psz_value += p_xml_entities[i].i_length+1; - break; - } - else if( cmp < 0 ) - i -= step; - else - i += step; + size_t olen = strlen (ent->psz_char); + memcpy (p_pos, ent->psz_char, olen); + p_pos += olen - 1; + psz_value += strlen (ent->psz_entity) + 1; } - if( cmp != 0 ) - { + else + { /* No match */ *p_pos = *psz_value; psz_value++; } @@ -469,47 +401,36 @@ void resolve_xml_special_chars( char *psz_value ) */ char *convert_xml_special_chars( const char *psz_content ) { - char *psz_temp = malloc( 6 * strlen( psz_content ) + 1 ); - const char *p_from = psz_content; + assert( psz_content ); + + const size_t len = strlen( psz_content ); + char *const psz_temp = malloc( 6 * len + 1 ); char *p_to = psz_temp; - while ( *p_from ) + if( psz_temp == NULL ) + return NULL; + for( size_t i = 0; i < len; i++ ) { - if ( *p_from == '<' ) - { - strcpy( p_to, "<" ); - p_to += 4; - } - else if ( *p_from == '>' ) - { - strcpy( p_to, ">" ); - p_to += 4; - } - else if ( *p_from == '&' ) - { - strcpy( p_to, "&" ); - p_to += 5; - } - else if( *p_from == '\"' ) - { - strcpy( p_to, """ ); - p_to += 6; - } - else if( *p_from == '\'' ) - { - strcpy( p_to, "'" ); - p_to += 6; - } - else + const char *str; + char c = psz_content[i]; + + switch ( c ) { - *p_to = *p_from; - p_to++; + case '\"': str = "quot"; break; + case '&': str = "amp"; break; + case '\'': str = "#39"; break; + case '<': str = "lt"; break; + case '>': str = "gt"; break; + default: + *(p_to++) = c; + continue; } - p_from++; + p_to += sprintf( p_to, "&%s;", str ); } - *p_to = '\0'; + *(p_to++) = '\0'; - return psz_temp; + p_to = realloc( psz_temp, p_to - psz_temp ); + return p_to ? p_to : psz_temp; /* cannot fail */ } /* Base64 encoding */ @@ -646,29 +567,57 @@ char *vlc_b64_decode( const char *psz_src ) return p_dst; } -/**************************************************************************** - * String formating functions - ****************************************************************************/ +/** + * Formats current time into a heap-allocated string. + * @param tformat time format (as with C strftime()) + * @return an allocated string (must be free()'d), or NULL on memory error. + */ char *str_format_time( const char *tformat ) { - char buffer[255]; time_t curtime; struct tm loctime; + if (strcmp (tformat, "") == 0) + return strdup (""); /* corner case w.r.t. strftime() return value */ + /* Get the current time. */ - curtime = time( NULL ); + time( &curtime ); /* Convert it to local time representation. */ localtime_r( &curtime, &loctime ); - strftime( buffer, 255, tformat, &loctime ); - return strdup( buffer ); + for (size_t buflen = strlen (tformat) + 32;; buflen += 32) + { + char *str = malloc (buflen); + if (str == NULL) + return NULL; + + size_t len = strftime (str, buflen, tformat, &loctime); + if (len > 0) + { + char *ret = realloc (str, len + 1); + return ret ? ret : str; /* <- this cannot fail */ + } + } + assert (0); +} + +static void format_duration (char *buf, size_t len, int64_t duration) +{ + lldiv_t d; + int sec; + + duration /= CLOCK_FREQ; + d = lldiv (duration, 60); + sec = d.rem; + d = lldiv (d.quot, 60); + snprintf (buf, len, "%02lld:%02d:%02d", d.quot, (int)d.rem, sec); } #define INSERT_STRING( string ) \ if( string != NULL ) \ { \ int len = strlen( string ); \ - dst = realloc( dst, i_size = i_size + len );\ + dst = xrealloc( dst, i_size = i_size + len );\ memcpy( (dst+d), string, len ); \ d += len; \ free( string ); \ @@ -683,11 +632,12 @@ char *str_format_time( const char *tformat ) #define INSERT_STRING_NO_FREE( string ) \ { \ int len = strlen( string ); \ - dst = realloc( dst, i_size = i_size + len );\ + dst = xrealloc( dst, i_size = i_size + len );\ memcpy( dst+d, string, len ); \ d += len; \ } -char *__str_format_meta( vlc_object_t *p_object, const char *string ) +#undef str_format_meta +char *str_format_meta( vlc_object_t *p_object, const char *string ) { const char *s = string; bool b_is_format = false; @@ -698,10 +648,8 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) if( !dst ) return NULL; int d = 0; - playlist_t *p_playlist = pl_Hold( p_object ); - input_thread_t *p_input = playlist_CurrentInput( p_playlist ); + input_thread_t *p_input = playlist_CurrentInput( pl_Get(p_object) ); input_item_t *p_item = NULL; - pl_Release( p_object ); if( p_input ) { p_item = input_GetItem(p_input); @@ -746,13 +694,13 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) case 'f': if( p_item && p_item->p_stats ) { - snprintf( buf, 10, "%d", + vlc_mutex_lock( &p_item->p_stats->lock ); + snprintf( buf, 10, "%"PRIi64, p_item->p_stats->i_displayed_pictures ); + vlc_mutex_unlock( &p_item->p_stats->lock ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'g': @@ -816,40 +764,31 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) case 'B': if( p_input ) { - snprintf( buf, 10, "%d", + snprintf( buf, 10, "%"PRId64, var_GetInteger( p_input, "bit-rate" )/1000 ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'C': if( p_input ) { - snprintf( buf, 10, "%d", + snprintf( buf, 10, "%"PRId64, var_GetInteger( p_input, "chapter" ) ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'D': if( p_item ) { mtime_t i_duration = input_item_GetDuration( p_item ); - sprintf( buf, "%02d:%02d:%02d", - (int)(i_duration/(3600000000)), - (int)((i_duration/(60000000))%60), - (int)((i_duration/1000000)%60) ); + format_duration (buf, sizeof (buf), i_duration); } else - { - sprintf( buf, b_empty_if_na ? "" : "--:--:--" ); - } + strcpy( buf, b_empty_if_na ? "" : "--:--:--" ); INSERT_STRING_NO_FREE( buf ); break; case 'F': @@ -861,29 +800,23 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) case 'I': if( p_input ) { - snprintf( buf, 10, "%d", + snprintf( buf, 10, "%"PRId64, var_GetInteger( p_input, "title" ) ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'L': if( p_item && p_input ) { mtime_t i_duration = input_item_GetDuration( p_item ); - int64_t i_time = p_input->i_time; - sprintf( buf, "%02d:%02d:%02d", - (int)( ( i_duration - i_time ) / 3600000000 ), - (int)( ( ( i_duration - i_time ) / 60000000 ) % 60 ), - (int)( ( ( i_duration - i_time ) / 1000000 ) % 60 ) ); + int64_t i_time = var_GetTime( p_input, "time" ); + format_duration( buf, sizeof(buf), + i_duration - i_time ); } else - { - sprintf( buf, b_empty_if_na ? "" : "--:--:--" ); - } + strcpy( buf, b_empty_if_na ? "" : "--:--:--" ); INSERT_STRING_NO_FREE( buf ); break; case 'N': @@ -911,20 +844,18 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) } else { - sprintf( buf, b_empty_if_na ? "" : "--.-%%" ); + snprintf( buf, 10, b_empty_if_na ? "" : "--.-%%" ); } INSERT_STRING_NO_FREE( buf ); break; case 'R': if( p_input ) { - int r = var_GetInteger( p_input, "rate" ); - snprintf( buf, 10, "%d.%d", r/1000, r%1000 ); + float f = var_GetFloat( p_input, "rate" ); + snprintf( buf, 10, "%.3f", f ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'S': @@ -934,23 +865,17 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) snprintf( buf, 10, "%d.%d", r/1000, (r/100)%10 ); } else - { - sprintf( buf, b_empty_if_na ? "" : "-" ); - } + strcpy( buf, b_empty_if_na ? "" : "-" ); INSERT_STRING_NO_FREE( buf ); break; case 'T': if( p_input ) { - sprintf( buf, "%02d:%02d:%02d", - (int)( p_input->i_time / ( 3600000000 ) ), - (int)( ( p_input->i_time / ( 60000000 ) ) % 60 ), - (int)( ( p_input->i_time / 1000000 ) % 60 ) ); + int64_t i_time = var_GetTime( p_input, "time" ); + format_duration( buf, sizeof(buf), i_time ); } else - { - sprintf( buf, b_empty_if_na ? "" : "--:--:--" ); - } + strcpy( buf, b_empty_if_na ? "" : "--:--:--" ); INSERT_STRING_NO_FREE( buf ); break; case 'U': @@ -1006,10 +931,11 @@ char *__str_format_meta( vlc_object_t *p_object, const char *string ) #undef INSERT_STRING #undef INSERT_STRING_NO_FREE +#undef str_format /** * Apply str format time and str format meta */ -char *__str_format( vlc_object_t *p_this, const char *psz_src ) +char *str_format( vlc_object_t *p_this, const char *psz_src ) { char *psz_buf1, *psz_buf2; psz_buf1 = str_format_time( psz_src ); @@ -1023,6 +949,10 @@ char *__str_format( vlc_object_t *p_this, const char *psz_src ) */ void filename_sanitize( char *str ) { +#if defined( WIN32 ) + char *str_base = str; +#endif + if( *str == '.' && (str[1] == '\0' || (str[1] == '.' && str[2] == '\0' ) ) ) { while( *str ) @@ -1033,6 +963,12 @@ void filename_sanitize( char *str ) return; } +#if defined( WIN32 ) + // Change leading spaces into underscores + while( *str && *str == ' ' ) + *str++ = '_'; +#endif + while( *str ) { switch( *str ) @@ -1054,6 +990,17 @@ void filename_sanitize( char *str ) } str++; } + +#if defined( WIN32 ) + // Change trailing spaces into underscores + str--; + while( str != str_base ) + { + if( *str != ' ' ) + break; + *str-- = '_'; + } +#endif } /** @@ -1061,16 +1008,10 @@ void filename_sanitize( char *str ) */ void path_sanitize( char *str ) { -#if 0 - /* - * Uncomment the two blocks to prevent /../ or /./, i'm not sure that we - * want to. - */ - char *prev = str - 1; -#endif #ifdef WIN32 /* check drive prefix if path is absolute */ - if( isalpha(*str) && (':' == *(str+1)) ) + if( (((unsigned char)(str[0] - 'A') < 26) + || ((unsigned char)(str[0] - 'a') < 26)) && (':' == str[1]) ) str += 2; #endif while( *str ) @@ -1079,37 +1020,212 @@ void path_sanitize( char *str ) if( *str == ':' ) *str = '_'; #elif defined( WIN32 ) - switch( *str ) + if( strchr( "*\"?:|<>", *str ) ) + *str = '_'; + if( *str == '/' ) + *str = DIR_SEP_CHAR; +#endif + str++; + } +} + +#include + +/** + * Convert a file path to an URI. + * If already an URI, return a copy of the string. + * @param path path to convert (or URI to copy) + * @param scheme URI scheme to use (default is auto: "file", "fd" or "smb") + * @return a nul-terminated URI string (use free() to release it), + * or NULL in case of error + */ +char *make_URI (const char *path, const char *scheme) +{ + if (path == NULL) + return NULL; + if (scheme == NULL && !strcmp (path, "-")) + return strdup ("fd://0"); // standard input + if (strstr (path, "://") != NULL) + return strdup (path); /* Already an URI */ + /* Note: VLC cannot handle URI schemes without double slash after the + * scheme name (such as mailto: or news:). */ + + char *buf; +#ifdef WIN32 + if (isalpha (path[0]) && (path[1] == ':')) + { + if (asprintf (&buf, "%s:///%c:", scheme, path[0]) == -1) + buf = NULL; + path += 2; + } + else +#endif + if (!strncmp (path, "\\\\", 2)) + { /* Windows UNC paths */ +#ifndef WIN32 + if (scheme != NULL) + return NULL; /* remote files not supported */ + + /* \\host\share\path -> smb://host/share/path */ + if (strchr (path + 2, '\\') != NULL) + { /* Convert backslashes to slashes */ + char *dup = strdup (path); + if (dup == NULL) + return NULL; + for (size_t i = 2; dup[i]; i++) + if (dup[i] == '\\') + dup[i] = DIR_SEP_CHAR; + + char *ret = make_URI (dup, scheme); + free (dup); + return ret; + } +# define SMB_SCHEME "smb" +#else + /* \\host\share\path -> file://host/share/path */ +# define SMB_SCHEME "file" +#endif + size_t hostlen = strcspn (path + 2, DIR_SEP); + + buf = malloc (sizeof (SMB_SCHEME) + 3 + hostlen); + if (buf != NULL) + snprintf (buf, sizeof (SMB_SCHEME) + 3 + hostlen, + SMB_SCHEME"://%s", path + 2); + path += 2 + hostlen; + } + else + if (path[0] != DIR_SEP_CHAR) + { /* Relative path: prepend the current working directory */ + char cwd[PATH_MAX]; + + if (getcwd (cwd, sizeof (cwd)) == NULL) /* FIXME: UTF8? */ + return NULL; + if (asprintf (&buf, "%s/%s", cwd, path) == -1) + return NULL; + char *ret = make_URI (buf, scheme); + free (buf); + return ret; + } + else + if (asprintf (&buf, "%s://", scheme ? scheme : "file") == -1) + buf = NULL; + if (buf == NULL) + return NULL; + + assert (path[0] == DIR_SEP_CHAR); + + /* Absolute file path */ + for (const char *ptr = path + 1;; ptr++) + { + size_t len = strcspn (ptr, DIR_SEP); + char *component = encode_URI_bytes (ptr, len); + if (component == NULL) { - case '*': - case '"': - case '?': - case ':': - case '|': - case '<': - case '>': - *str = '_'; + free (buf); + return NULL; } + char *uri; + int val = asprintf (&uri, "%s/%s", buf, component); + free (component); + free (buf); + if (val == -1) + return NULL; + buf = uri; + ptr += len; + if (*ptr == '\0') + return buf; + } +} + +/** + * Tries to convert an URI to a local (UTF-8-encoded) file path. + * @param url URI to convert + * @return NULL on error, a nul-terminated string otherwise + * (use free() to release it) + */ +char *make_path (const char *url) +{ + char *ret = NULL; + char *end; + + char *path = strstr (url, "://"); + if (path == NULL) + return NULL; /* unsupported scheme or invalid syntax */ + + end = memchr (url, '/', path - url); + size_t schemelen = ((end != NULL) ? end : path) - url; + path += 3; /* skip "://" */ + + /* Remove HTML anchor if present */ + end = strchr (path, '#'); + if (end) + path = strndup (path, end - path); + else + path = strdup (path); + if (unlikely(path == NULL)) + return NULL; /* boom! */ + + /* Decode path */ + decode_URI (path); + + if (schemelen == 4 && !strncasecmp (url, "file", 4)) + { +#if (DIR_SEP_CHAR != '/') + for (char *p = strchr (path, '/'); p; p = strchr (p + 1, '/')) + *p = DIR_SEP_CHAR; +#endif + /* Leading slash => local path */ + if (*path == DIR_SEP_CHAR) +#if !defined (WIN32) || defined (UNDER_CE) + return path; +#else + return memmove (path, path + 1, strlen (path + 1) + 1); #endif -#if 0 - if( *str == '/' + + /* Local path disguised as a remote one (MacOS X) */ + if (!strncasecmp (path, "localhost"DIR_SEP, 10)) + return memmove (path, path + 9, strlen (path + 9) + 1); + #ifdef WIN32 - || *str == '\\' + if (*path && asprintf (&ret, "\\\\%s", path) == -1) + ret = NULL; #endif - ) + /* non-local path :-( */ + } + else + if (schemelen == 2 && !strncasecmp (url, "fd", 2)) + { + int fd = strtol (path, &end, 0); + + if (*end) + goto out; + +#ifndef WIN32 + switch (fd) { - if( str - prev == 2 && prev[1] == '.' ) - { - prev[1] = '.'; - } - else if( str - prev == 3 && prev[1] == '.' && prev[2] == '.' ) - { - prev[1] = '_'; - prev[2] = '_'; - } - prev = str; + case 0: + ret = strdup ("/dev/stdin"); + break; + case 1: + ret = strdup ("/dev/stdout"); + break; + case 2: + ret = strdup ("/dev/stderr"); + break; + default: + if (asprintf (&ret, "/dev/fd/%d", fd) == -1) + ret = NULL; } +#else + /* XXX: Does this work on WinCE? */ + if (fd < 2) + ret = strdup ("CON"); + else + ret = NULL; #endif - str++; } + +out: + free (path); + return ret; /* unknown scheme */ }