strdup -> xstrdup
authorAndre Heinecke <aheinecke@intevation.de>
Mon, 23 Jul 2018 11:04:34 +0000 (13:04 +0200)
committerAndre Heinecke <aheinecke@intevation.de>
Mon, 23 Jul 2018 11:04:34 +0000 (13:04 +0200)
--

src/common.cpp
src/cpphelp.cpp
src/mapihelp.cpp
src/memdbg.cpp
src/mimemaker.cpp
src/mlang-charset.cpp
src/oomhelp.cpp
src/rfc2047parse.c
src/w32-gettext.cpp
src/wks-helper.cpp

index 689a0b9..9c0d6e2 100644 (file)
@@ -198,7 +198,7 @@ read_w32_registry_string (const char *root, const char *dir, const char *name)
     {
       return nullptr;
     }
-  return strdup (ret.c_str ());
+  return xstrdup (ret.c_str ());
 }
 
 /* Return the data dir used for forms etc.   Returns NULL on error. */
@@ -468,7 +468,7 @@ get_tmp_outfile (wchar_t *name, HANDLE *outHandle)
       log_debug_w32 (-1, "%s:%s: Failed to open candidate '%s'",
                      SRCNAME, __func__, outName.c_str());
 
-      char *outNameC = strdup (outName.c_str());
+      char *outNameC = xstrdup (outName.c_str());
 
       const auto lastBackslash = strrchr (outNameC, '\\');
       if (!lastBackslash)
index 2dc25a4..5912f81 100644 (file)
@@ -76,7 +76,7 @@ vector_to_cArray(const std::vector<std::string> &vec)
   char ** ret = (char**) xmalloc (sizeof (char*) * (vec.size() + 1));
   for (size_t i = 0; i < vec.size(); i++)
     {
-      ret[i] = strdup (vec[i].c_str());
+      ret[i] = xstrdup (vec[i].c_str());
     }
   ret[vec.size()] = NULL;
   return ret;
index afe6d7f..301f692 100644 (file)
@@ -326,7 +326,7 @@ mapi_get_uid (LPMESSAGE msg)
     }
   else if (PROP_TYPE (propval->ulPropTag) == PT_STRING8)
     {
-      ret = strdup (propval->Value.lpszA);
+      ret = xstrdup (propval->Value.lpszA);
       log_debug ("%s:%s: Fund uuid in MAPI for %p",
                  SRCNAME, __func__, msg);
     }
index 0a50513..aabf2a8 100644 (file)
@@ -53,7 +53,7 @@ register_name (void *obj, const char *nameSuggestion)
 
   if (!name && nameSuggestion)
     {
-      name = strdup (nameSuggestion);
+      name = xstrdup (nameSuggestion);
       suggestionUsed = true;
     }
   if (!name)
index 4dd55a9..216acdf 100644 (file)
@@ -176,7 +176,7 @@ create_mapi_attachment (LPMESSAGE message, sink_t sink,
 
   /* We better insert a short filename. */
   prop.ulPropTag = PR_ATTACH_FILENAME_A;
-  prop.Value.lpszA = strdup (MIMEATTACHFILENAME);
+  prop.Value.lpszA = xstrdup (MIMEATTACHFILENAME);
   hr = HrSetOneProp ((LPMAPIPROP)att, &prop);
   xfree (prop.Value.lpszA);
   if (hr)
@@ -197,8 +197,8 @@ create_mapi_attachment (LPMESSAGE message, sink_t sink,
   if (!hr)
     {
       prop.ulPropTag = PR_ATTACH_MIME_TAG_A;
-      prop.Value.lpszA = overrideMimeTag ? strdup (overrideMimeTag) :
-                         strdup ("multipart/signed");
+      prop.Value.lpszA = overrideMimeTag ? xstrdup (overrideMimeTag) :
+                         xstrdup ("multipart/signed");
       if (overrideMimeTag)
         {
           log_debug ("%s:%s: using override mimetag: %s\n",
@@ -1150,11 +1150,11 @@ finalize_message (LPMESSAGE message, mapi_attach_item_t *att_table,
   prop.ulPropTag = PR_MESSAGE_CLASS_A;
   if (encrypt)
     {
-      prop.Value.lpszA = strdup ("IPM.Note.InfoPathForm.GpgOL.SMIME.MultipartSigned");
+      prop.Value.lpszA = xstrdup ("IPM.Note.InfoPathForm.GpgOL.SMIME.MultipartSigned");
     }
   else
     {
-      prop.Value.lpszA = strdup ("IPM.Note.InfoPathForm.GpgOLS.SMIME.MultipartSigned");
+      prop.Value.lpszA = xstrdup ("IPM.Note.InfoPathForm.GpgOLS.SMIME.MultipartSigned");
     }
 
   if (!is_inline)
index eb4cc85..e9b327a 100644 (file)
@@ -48,7 +48,7 @@ char *ansi_charset_to_utf8 (const char *charset, const char *input,
     {
       log_debug ("%s:%s: No charset / codepage returning plain.",
                  SRCNAME, __func__);
-      return strdup (input);
+      return xstrdup (input);
     }
 
   CoCreateInstance(CLSID_CMultiLanguage, NULL, CLSCTX_INPROC_SERVER,
@@ -84,7 +84,7 @@ char *ansi_charset_to_utf8 (const char *charset, const char *input,
           log_error ("%s:%s: Failed to find charset for: %s",
                      SRCNAME, __func__, charset);
           gpgol_release (multilang);
-          return strdup(input);
+          return xstrdup (input);
         }
       enc = (mime_info.uiInternetEncoding == 0) ? mime_info.uiCodePage :
                                                   mime_info.uiInternetEncoding;
index 1a11560..297fe0b 100644 (file)
@@ -1918,7 +1918,7 @@ generate_uid ()
   unsigned char *str;
   UuidToStringA (&uuid, &str);
 
-  char *ret = strdup ((char*)str);
+  char *ret = xstrdup ((char*)str);
   RpcStringFreeA (&str);
 
   return ret;
@@ -1956,7 +1956,7 @@ get_unique_id (LPDISPATCH mail, int create, const char *uuid)
     }
   else
     {
-      newuid = strdup (uuid);
+      newuid = xstrdup (uuid);
     }
   int ret = put_pa_string (mail, GPGOL_UID_DASL, newuid);
 
index 56e5dba..10fd7fc 100644 (file)
@@ -652,7 +652,7 @@ rfc2047_parse (const char *input)
 {
   char *decoded;
   if (!input)
-    return strdup ("");
+    return xstrdup ("");
 
   log_debug ("%s:%s: Input: \"%s\"",
              SRCNAME, __func__, input);
@@ -665,7 +665,7 @@ rfc2047_parse (const char *input)
   if (!decoded || !strlen (decoded))
     {
       xfree (decoded);
-      return strdup (input);
+      return xstrdup (input);
     }
   return decoded;
 }
index b89bc79..430eafa 100644 (file)
@@ -1596,7 +1596,7 @@ get_string (struct loaded_domain *domain, u32 idx, int utf8)
       domain->mapped[idx] = 1;
 
       plen = strlen (p);
-      buf = utf8 ? strdup (p) : utf8_to_native (p);
+      buf = utf8 ? xstrdup (p) : utf8_to_native (p);
       buflen = strlen (buf);
       if (buflen <= plen)
         strcpy (p, buf);
index 109ecab..817c487 100644 (file)
@@ -300,7 +300,7 @@ WKSHelper::start_check (const std::string &mbox, bool forced) const
   log_debug ("%s:%s: WKSHelper starting check",
              SRCNAME, __func__);
   /* Start the actual work that can be done in a background thread. */
-  CloseHandle (CreateThread (nullptr, 0, do_check, strdup (mbox.c_str ()), 0,
+  CloseHandle (CreateThread (nullptr, 0, do_check, xstrdup (mbox.c_str ()), 0,
                              nullptr));
   return;
 }
@@ -386,7 +386,7 @@ WKSHelper::allow_notify (int sleepTimeMS) const
       if (pair.second == ConfirmationSeen ||
           pair.second == NeedsPublish)
         {
-          auto *args = new std::pair<char *, int> (strdup (pair.first.c_str()),
+          auto *args = new std::pair<char *, int> (xstrdup (pair.first.c_str()),
                                                    sleepTimeMS);
           CloseHandle (CreateThread (nullptr, 0, do_notify,
                                      args, 0,