Merge pull request #1715 from dbwiddis/fix-name-path

Fix UdevDevice.getSysname() calling udev_device_get_syspath

Author: dbwiddis

CHANGES.md

@@ -13,6 +13,7 @@ Features
 Bug Fixes
 ---------
 * [#1644](https://github.com/java-native-access/jna/issues/1644): Fix bug in VARDESC and TYPEDESC causing an illegal memory access - [@lwahonen](https://github.com/lwahonen)
+* [#1715](https://github.com/java-native-access/jna/pull/1715): Fix `UdevDevice.getSysname()` calling `udev_device_get_syspath` instead of `udev_device_get_sysname` - [@dbwiddis](https://github.com/dbwiddis).
 
 Release 5.18.1
 ==============

contrib/platform/src/com/sun/jna/platform/linux/Udev.java

@@ -233,7 +233,7 @@ public String getSyspath() {
          * @return a string that describes the sysname. On failure, may return NULL.
          */
         public String getSysname() {
-            return INSTANCE.udev_device_get_syspath(this);
+            return INSTANCE.udev_device_get_sysname(this);
         }
 
         /**

contrib/platform/test/com/sun/jna/platform/linux/UdevTest.java

@@ -111,7 +111,9 @@ public void testEnumerateDevices() {
                                 assertEquals(String.format("DevType mismatch (%s)", devnode), devType, device.getDevtype());
                                 assertEquals(String.format("Subsystem mismatch (%s)", devnode), "block", device.getSubsystem());
                                 assertEquals(String.format("Syspath mismatch (%s)", devnode), syspath, device.getSyspath());
-                                assertTrue(String.format("Syspath should end with name (%s)", devnode), syspath.endsWith(device.getSysname()));
+                                String sysname = device.getSysname();
+                                assertTrue(String.format("Syspath should end with name (%s)", devnode), syspath.endsWith(sysname));
+                                assertTrue(String.format("Sysname should be shorter than syspath (%s)", devnode), sysname.length() < syspath.length());
                             }
                         } finally {
                             // Release the reference and iterate to the next device