usd: Can't call usd_FileStandard* with NULL

It doesn't make sense to call usd_FileStandard{Input,Output} with
a NULL usd_handle_t (and doing so would crash later in the
function), so don't check for attempts to do so.

Caught by clang-analyzer

Change-Id: I03c0b627056108fe0d6f78d8028323cc4ed74758
Reviewed-on: http://gerrit.openafs.org/9151
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

diff --git a/src/usd/usd_file.c b/src/usd/usd_file.c
index f66df08aceb69b6813e98a7df27267121a672baa..06c6a4cd8084df61362a2cbe6bb87b04573c7dff 100644 (file)
--- a/src/usd/usd_file.c
+++ b/src/usd/usd_file.c
@@ -392,9 +392,6 @@ usd_FileStandardInput(usd_handle_t * usdP)
 {
     usd_handle_t usd;
 
-    if (usdP)
-       *usdP = NULL;
-
     usd = calloc(1, sizeof(*usd));
     usd->handle = (void *)((unsigned long)0);
     usd->read = usd_FileRead;
@@ -420,9 +417,6 @@ usd_FileStandardOutput(usd_handle_t * usdP)
 {
     usd_handle_t usd;
 
-    if (usdP)
-       *usdP = NULL;
-
     usd = calloc(1, sizeof(*usd));
     usd->handle = (void *)((unsigned long)1);
     usd->read = usd_FileRead;