Bug 1645479 - Use SECITEM_CopyItem instead of SECITEM_MakeItem in sec…

…util.c. r=jcj

This patch converts a call to `SECITEM_MakeItem` to use `SECITEM_CopyItem` instead. Using the former works fine in NSS CI, but causes build failures in mozilla-central due to differences in how both symbols are exported (i.e. when folding nssutil into nss).

Differential Revision: https://phabricator.services.mozilla.com/D79525

--HG--
extra : moz-landing-system : lando

cmd/lib/secutil.c

@@ -4200,10 +4200,10 @@ readPSK(const char *arg, SECItem *psk, SECItem *label)
         }
         rv = SECSuccess;
     } else {
-        const PRUint8 defaultLabel[] = { 'C', 'l', 'i', 'e', 'n', 't', '_',
-                                         'i', 'd', 'e', 'n', 't', 'i', 't', 'y' };
-        rv = SECITEM_MakeItem(NULL, &labelItem, defaultLabel,
-                              sizeof(defaultLabel));
+        PRUint8 defaultLabel[] = { 'C', 'l', 'i', 'e', 'n', 't', '_',
+                                   'i', 'd', 'e', 'n', 't', 'i', 't', 'y' };
+        SECItem src = { siBuffer, defaultLabel, sizeof(defaultLabel) };
+        rv = SECITEM_CopyItem(NULL, &labelItem, &src);
     }
     if (rv == SECSuccess) {
         *label = labelItem;