]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
secret: Fix error for private secrets
authorMartin Kletzander <mkletzan@redhat.com>
Tue, 18 Sep 2012 10:01:46 +0000 (12:01 +0200)
committerMartin Kletzander <mkletzan@redhat.com>
Tue, 18 Sep 2012 14:41:20 +0000 (16:41 +0200)
When trying to get the value of a private secret, the code used
'operation denied' error. That error is specified as a error for
read-only connections trying to perform denied operation. The
following error seems more accurate.

To compare the difference:
 - BEFORE
error: operation secret is private forbidden for read only access

 - AFTER
error: Invalid secret: secret is private

src/secret/secret_driver.c

index 2ad66dd3e8cb801c7a81b22d3ea184695425510a..67c7323b1f19991fde10408604da5c406d67e65a 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * secret_driver.c: local driver for secret manipulation API
  *
- * Copyright (C) 2009-2011 Red Hat, Inc.
+ * Copyright (C) 2009-2012 Red Hat, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -990,7 +990,7 @@ secretGetValue(virSecretPtr obj, size_t *value_size, unsigned int flags,
 
     if ((internalFlags & VIR_SECRET_GET_VALUE_INTERNAL_CALL) == 0 &&
         secret->def->private) {
-        virReportError(VIR_ERR_OPERATION_DENIED, "%s",
+        virReportError(VIR_ERR_INVALID_SECRET, "%s",
                        _("secret is private"));
         goto cleanup;
     }