luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-02-14 06:17:15 +01:00
Code Issues Packages Projects Releases Wiki Activity

Revert "[MacOS] Fallback to CFStringGetCSTring if CFStringGetCStringPtr fails."

Browse Source 
This reverts commit d5a9ce69c46a907af8ed3af0020d2a409faa3751.
Partial resolution of bug 789894
This commit is contained in:
John Ralls 2017-11-01 18:00:49 -07:00
parent 52b3e434d8
commit d74a69826f
2 changed files with 4 additions and 64 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
gio/gosxappinfo.c
View File

@ -175,44 +175,14 @@ static gchar *
create_cstr_from_cfstring (CFStringRef str) create_cstr_from_cfstring (CFStringRef str)
{ {
const gchar *cstr; const gchar *cstr;
CFIndex length = CFStringGetLength (str);
char *buffer = NULL;
if (str == NULL) if (str == NULL)
return NULL; return NULL;
cstr = CFStringGetCStringPtr (str, kCFStringEncodingUTF8); cstr = CFStringGetCStringPtr (str, kCFStringEncodingUTF8);
/* CFStringGetCStringPtr returns "NULL if the internal storage of
* theString does not allow [a pointer] to be returned efficiently".
* (Apple's docs don't say what that means). In that case we must
* use CFStringGetCString as a fallback.
*/
if (cstr != NULL)
{
CFRelease (str);
return g_strdup (cstr);
}
buffer = g_malloc0 (length + 1);
/* Start off with a buffer size sufficient for the most likely case
* that the CFString is ASCII so the UTF8 representation will be 1
* byte per code point. Keep trying up to 4 bytes per code point,
* the max allowed by RFC3629.
*/
for (int i = 1; i <= 4; ++i)
{
if (CFStringGetCString (str, buffer, length, kCFStringEncodingUTF8))
{
CFRelease (str);
return buffer;
}
length += length;
buffer = g_realloc (buffer, length + 1);
}
g_free (buffer);
CFRelease (str); CFRelease (str);
return NULL;
return g_strdup (cstr);
} }
static char * static char *

34
gio/gosxcontenttype.c
View File

@ -53,44 +53,14 @@ static gchar *
create_cstr_from_cfstring (CFStringRef str) create_cstr_from_cfstring (CFStringRef str)
{ {
const gchar *cstr; const gchar *cstr;
CFIndex length = CFStringGetLength (str);
char *buffer = NULL;
if (str == NULL) if (str == NULL)
return NULL; return NULL;
cstr = CFStringGetCStringPtr (str, kCFStringEncodingUTF8); cstr = CFStringGetCStringPtr (str, kCFStringEncodingUTF8);
/* CFStringGetCStringPtr returns "NULL if the internal storage of
* theString does not allow [a pointer] to be returned efficiently".
* (Apple's docs don't say what that means). In that case we must
* use CFStringGetCString as a fallback.
*/
if (cstr != NULL)
{
CFRelease (str);
return g_strdup (cstr);
}
buffer = g_malloc0 (length + 1);
/* Start off with a buffer size sufficient for the most likely case
* that the CFString is ASCII so the UTF8 representation will be 1
* byte per code point. Keep trying up to 4 bytes per code point,
* the max allowed by RFC3629.
*/
for (int i = 1; i <= 4; ++i)
{
if (CFStringGetCString (str, buffer, length, kCFStringEncodingUTF8))
{
CFRelease (str);
return buffer;
}
length += length;
buffer = g_realloc (buffer, length + 1);
}
g_free (buffer);
CFRelease (str); CFRelease (str);
return NULL;
return g_strdup (cstr);
} }
/*< internal > /*< internal >