transaction: Properly handle nil bytes in binary transactions

If returned binaries are nil, we should pass them as nil and not as an empty bytes array.
msteinert · Oct 12, 2023 · 0bdfc69 · 0bdfc69
1 parent beeb5ff
commit 0bdfc69
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/transaction.go b/transaction.go
@@ -188,6 +188,8 @@ func cbPAMConv(s C.int, msg *C.char, c C.uintptr_t) (*C.char, C.int) {
 			bytes, err := cb.RespondPAMBinary(BinaryPointer(msg))
 			if err != nil {
 				return nil, ConvAgain.toC()
+			} else if bytes == nil {
+				return nil, Success.toC()
 			}
 			return (*C.char)(C.CBytes(bytes)), Success.toC()
 		} else {