Re: [PATCH] Allow byte[] arrays to be set as a secretValue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/09/2012 03:58 PM, Eric Blake wrote:
On 03/09/2012 07:23 AM, Wido den Hollander wrote:

Signed-off-by: Wido den Hollander<wido@xxxxxxxxx>
---
  src/main/java/org/libvirt/Secret.java      |   11 +++++++++++
  src/main/java/org/libvirt/jna/Libvirt.java |    1 +
  2 files changed, 12 insertions(+), 0 deletions(-)

If you will do:

git config format.subjectprefix 'PATCH libvirt-jaba'

it will make it easier to identify when you are patching the java bindings.

I'll do! Will re-send the correct patch shortly.



diff --git a/src/main/java/org/libvirt/Secret.java b/src/main/java/org/libvirt/Secret.java
index e536cf4..48f7895 100644
--- a/src/main/java/org/libvirt/Secret.java
+++ b/src/main/java/org/libvirt/Secret.java
@@ -146,6 +146,17 @@ public class Secret {
      }

      /**
+     * Sets the value of the secret
+     *
+     * @return 0 on success, -1 on failure.
+     */
+    public int setValue(byte[] value) throws LibvirtException {
+        int returnValue = libvirt.virSecretSetValue(VSP, value, new NativeLong(value.length), 0);
+        processError();
+        return returnValue;
+    }

It looks like the last commit to libvirt-java.git was more than a year
ago; there's a lot of missing bindings.  Would you be willing to step in
and provide even more patches on this front?

It's indeed pretty old. I ran into this issue when working with Ceph/RBD. It uses a binary key and that won't go into a String.

I'm currently not available to pick this up further. In the future I might run into it when working with CloudStack, but at this point I'm a bit busy.

> Meanwhile, this patch looks okay to me, but I've never contributed to libvirt-java; I'll wait
to see if someone who has actually participated in this project has
comments.


Great!

Wido

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]