[PATCH] tests: Fix runtime error in test-authz-pam

Stefan Weil posted 1 patch 3 years, 3 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210128171127.45808-1-sw@weilnetz.de
Maintainers: "Daniel P. Berrangé" <berrange@redhat.com>
tests/test-authz-pam.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
[PATCH] tests: Fix runtime error in test-authz-pam
Posted by Stefan Weil 3 years, 3 months ago
A test with sanitizers on macOS shows this error:

    authz/pamacct.c:50:25: runtime error: null pointer passed as argument 1, which is declared to never be null
    /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/security/pam_appl.h:56:2: note: nonnull attribute specified here

Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
 tests/test-authz-pam.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/tests/test-authz-pam.c b/tests/test-authz-pam.c
index 1baeadee03..4fe1ef2603 100644
--- a/tests/test-authz-pam.c
+++ b/tests/test-authz-pam.c
@@ -28,7 +28,7 @@
 static bool failauth;
 
 /*
- * These two functions are exported by libpam.so.
+ * These three functions are exported by libpam.so.
  *
  * By defining them again here, our impls are resolved
  * by the linker instead of those in libpam.so
@@ -50,6 +50,7 @@ pam_start(const char *service_name, const char *user,
         failauth = false;
     }
 
+    *pamh = (pam_handle_t *)0xbadeaffe;
     return PAM_SUCCESS;
 }
 
@@ -65,6 +66,13 @@ pam_acct_mgmt(pam_handle_t *pamh, int flags)
 }
 
 
+int
+pam_end(pam_handle_t *pamh, int status)
+{
+    return PAM_SUCCESS;
+}
+
+
 static void test_authz_unknown_service(void)
 {
     Error *local_err = NULL;
-- 
2.24.3 (Apple Git-128)


Re: [PATCH] tests: Fix runtime error in test-authz-pam
Posted by Daniel P. Berrangé 3 years, 3 months ago
On Thu, Jan 28, 2021 at 06:11:27PM +0100, Stefan Weil wrote:
> A test with sanitizers on macOS shows this error:
> 
>     authz/pamacct.c:50:25: runtime error: null pointer passed as argument 1, which is declared to never be null
>     /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/security/pam_appl.h:56:2: note: nonnull attribute specified here
> 
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> ---
>  tests/test-authz-pam.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)


Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

and will add to my queue.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|