Remove sentences in RETURN VALUES sections saying that functions with

void return types 'return no value'. This is obvious and therefore
unneccessary to mention.

We spare rewind(3)'s sentence because espie@ pointed out that it's a
warning - the function masks a potential error.

This commit also adds a sentence to X509_free clarifying that it's
NULL-safe. This bit was discussed with doug@.

ok martijn@, sentiment supported by schwarze@

17 files changed, 14 insertions, 56 deletions

diff --git a/lib/libc/stdlib/hcreate.3 b/lib/libc/stdlib/hcreate.3
index ea264b57c84..610d2f3f88f 100644
--- a/lib/libc/stdlib/hcreate.3
+++ b/lib/libc/stdlib/hcreate.3
@@ -1,4 +1,4 @@
-.\" 	$OpenBSD: hcreate.3,v 1.6 2010/07/28 09:00:20 ray Exp $
+.\" 	$OpenBSD: hcreate.3,v 1.7 2016/03/12 21:31:22 mmcc Exp $
 .\" 	$NetBSD: hcreate.3,v 1.8 2010/05/01 06:18:03 jruoho Exp $
 .\"
 .\" Copyright (c) 1999 The NetBSD Foundation, Inc.
@@ -28,7 +28,7 @@
 .\" ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 .\" POSSIBILITY OF SUCH DAMAGE.
 .\"
-.Dd $Mdocdate: July 28 2010 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt HCREATE 3
 .Os
 .Sh NAME
@@ -159,11 +159,6 @@ Otherwise, a value of 0 is returned and
 .Va errno
 is set to indicate the error.
 .Pp
-The
-.Fn hdestroy
-functions
-returns no value.
-.Pp
 If successful, the
 .Fn hsearch
 function returns a pointer to a hash table entry matching
diff --git a/lib/libc/stdlib/malloc.3 b/lib/libc/stdlib/malloc.3
index 2e820047d01..c0be3a177d4 100644
--- a/lib/libc/stdlib/malloc.3
+++ b/lib/libc/stdlib/malloc.3
@@ -30,9 +30,9 @@
 .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
 .\" SUCH DAMAGE.
 .\"
-.\"	$OpenBSD: malloc.3,v 1.93 2016/02/05 15:09:09 schwarze Exp $
+.\"	$OpenBSD: malloc.3,v 1.94 2016/03/12 21:31:22 mmcc Exp $
 .\"
-.Dd $Mdocdate: February 5 2016 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt MALLOC 3
 .Os
 .Sh NAME
@@ -173,10 +173,6 @@ and set
 .Va errno
 to
 .Er ENOMEM .
-.Pp
-The
-.Fn free
-function returns no value.
 .Sh IDIOMS
 Consider
 .Fn calloc
diff --git a/lib/libc/stdlib/qsort.3 b/lib/libc/stdlib/qsort.3
index 36eaed879b5..f753777780f 100644
--- a/lib/libc/stdlib/qsort.3
+++ b/lib/libc/stdlib/qsort.3
@@ -29,9 +29,9 @@
 .\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
 .\" SUCH DAMAGE.
 .\"
-.\"	$OpenBSD: qsort.3,v 1.18 2015/01/29 01:46:31 schwarze Exp $
+.\"	$OpenBSD: qsort.3,v 1.19 2016/03/12 21:31:22 mmcc Exp $
 .\"
-.Dd $Mdocdate: January 29 2015 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt QSORT 3
 .Os
 .Sh NAME
@@ -143,10 +143,6 @@ which is faster than
 .Fn heapsort .
 Memory availability and pre-existing order in the data can make this untrue.
 .Sh RETURN VALUES
-The
-.Fn qsort
-function returns no value.
-.Pp
 .Rv -std heapsort mergesort
 .Sh ERRORS
 The
diff --git a/lib/libc/stdlib/tsearch.3 b/lib/libc/stdlib/tsearch.3
index 46d0bba10c1..2027f163e43 100644
--- a/lib/libc/stdlib/tsearch.3
+++ b/lib/libc/stdlib/tsearch.3
@@ -1,4 +1,4 @@
-.\" $OpenBSD: tsearch.3,v 1.19 2013/06/05 03:39:23 tedu Exp $
+.\" $OpenBSD: tsearch.3,v 1.20 2016/03/12 21:31:22 mmcc Exp $
 .\"
 .\" Copyright (c) 1997 Todd C. Miller <Todd.Miller@courtesan.com>
 .\"
@@ -14,7 +14,7 @@
 .\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 .\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 .\"
-.Dd $Mdocdate: June 5 2013 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt TSEARCH 3
 .Os
 .Sh NAME
@@ -111,10 +111,6 @@ if
 is
 .Dv NULL
 or the datum cannot be found.
-.Pp
-The
-.Fn twalk
-function returns no value.
 .Sh SEE ALSO
 .Xr bsearch 3 ,
 .Xr lsearch 3
diff --git a/lib/libcrypto/man/ASN1_OBJECT_new.3 b/lib/libcrypto/man/ASN1_OBJECT_new.3
index 5b81b2bded3..526e1d869bf 100644
--- a/lib/libcrypto/man/ASN1_OBJECT_new.3
+++ b/lib/libcrypto/man/ASN1_OBJECT_new.3
@@ -1,4 +1,4 @@
-.Dd $Mdocdate: September 9 2015 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt ASN1_OBJECT_NEW 3
 .Os
 .Sh NAME
@@ -47,9 +47,6 @@ returns
 and sets an error code that can be obtained by
 .Xr ERR_get_error 3 .
 Otherwise it returns a pointer to the newly allocated structure.
-.Pp
-.Fn ASN1_OBJECT_free
-returns no value.
 .Sh SEE ALSO
 .Xr d2i_ASN1_OBJECT 3 ,
 .Xr ERR_get_error 3 ,
diff --git a/lib/libcrypto/man/BUF_MEM_new.3 b/lib/libcrypto/man/BUF_MEM_new.3
index f87e4f82577..10258b7aaf4 100644
--- a/lib/libcrypto/man/BUF_MEM_new.3
+++ b/lib/libcrypto/man/BUF_MEM_new.3
@@ -1,4 +1,4 @@
-.Dd $Mdocdate: September 22 2015 $
+.Dd $Mdocdate: March 12 2016 $
 .Dt BUF_MEM_NEW 3
 .Os
 .Sh NAME
@@ -89,9 +89,6 @@ returns the buffer or
 .Dv NULL
 on error.
 .Pp
-.Fn BUF_MEM_free
-returns no value.
-.Pp
 .Fn BUF_MEM_grow
 returns zero on error or the new size (i.e.
 .Fa len Ns ).
diff --git a/lib/libssl/src/doc/crypto/DH_new.pod b/lib/libssl/src/doc/crypto/DH_new.pod
index d6c3ca82b5f..0fdb7b9680c 100644
--- a/lib/libssl/src/doc/crypto/DH_new.pod
+++ b/lib/libssl/src/doc/crypto/DH_new.pod
@@ -25,8 +25,6 @@ If the allocation fails, DH_new() returns B<NULL> and sets an error code that
 can be obtained by L<ERR_get_error(3)|ERR_get_error(3)>. Otherwise it returns a
 pointer to the newly allocated structure.
 
-DH_free() returns no value.
-
 =head1 SEE ALSO
 
 L<dh(3)|dh(3)>, L<ERR_get_error(3)|ERR_get_error(3)>,
diff --git a/lib/libssl/src/doc/crypto/DH_set_method.pod b/lib/libssl/src/doc/crypto/DH_set_method.pod
index 2a4c2735cb7..d82fe7377a2 100644
--- a/lib/libssl/src/doc/crypto/DH_set_method.pod
+++ b/lib/libssl/src/doc/crypto/DH_set_method.pod
@@ -89,8 +89,6 @@ DH_set_default_method() is used.
 DH_OpenSSL() and DH_get_default_method() return pointers to the respective
 B<DH_METHOD>s.
 
-DH_set_default_method() returns no value.
-
 DH_set_method() returns non-zero if the provided B<meth> was successfully set as
 the method for B<dh> (including unloading the ENGINE handle if the previous
 method was supplied by an ENGINE).
diff --git a/lib/libssl/src/doc/crypto/DSA_SIG_new.pod b/lib/libssl/src/doc/crypto/DSA_SIG_new.pod
index 3ac61400381..77aa649db05 100644
--- a/lib/libssl/src/doc/crypto/DSA_SIG_new.pod
+++ b/lib/libssl/src/doc/crypto/DSA_SIG_new.pod
@@ -26,8 +26,6 @@ error code that can be obtained by
 L<ERR_get_error(3)|ERR_get_error(3)>. Otherwise it returns a pointer
 to the newly allocated structure.
 
-DSA_SIG_free() returns no value.
-
 =head1 SEE ALSO
 
 L<dsa(3)|dsa(3)>, L<ERR_get_error(3)|ERR_get_error(3)>,
diff --git a/lib/libssl/src/doc/crypto/DSA_new.pod b/lib/libssl/src/doc/crypto/DSA_new.pod
index 48e9b82a09c..e1e30b9a079 100644
--- a/lib/libssl/src/doc/crypto/DSA_new.pod
+++ b/lib/libssl/src/doc/crypto/DSA_new.pod
@@ -27,8 +27,6 @@ code that can be obtained by
 L<ERR_get_error(3)|ERR_get_error(3)>. Otherwise it returns a pointer
 to the newly allocated structure.
 
-DSA_free() returns no value.
-
 =head1 SEE ALSO
 
 L<dsa(3)|dsa(3)>, L<ERR_get_error(3)|ERR_get_error(3)>,
diff --git a/lib/libssl/src/doc/crypto/DSA_set_method.pod b/lib/libssl/src/doc/crypto/DSA_set_method.pod
index 31f444d7512..bc57a3e8e2d 100644
--- a/lib/libssl/src/doc/crypto/DSA_set_method.pod
+++ b/lib/libssl/src/doc/crypto/DSA_set_method.pod
@@ -103,8 +103,6 @@ struct
 DSA_OpenSSL() and DSA_get_default_method() return pointers to the respective
 B<DSA_METHOD>s.
 
-DSA_set_default_method() returns no value.
-
 DSA_set_method() returns non-zero if the provided B<meth> was successfully set
 as the method for B<dsa> (including unloading the ENGINE handle if the previous
 method was supplied by an ENGINE).
diff --git a/lib/libssl/src/doc/crypto/ERR_load_strings.pod b/lib/libssl/src/doc/crypto/ERR_load_strings.pod
index 5acdd0edbc5..e9c5cf0fc56 100644
--- a/lib/libssl/src/doc/crypto/ERR_load_strings.pod
+++ b/lib/libssl/src/doc/crypto/ERR_load_strings.pod
@@ -38,7 +38,7 @@ to user libraries at runtime.
 
 =head1 RETURN VALUE
 
-ERR_load_strings() returns no value. ERR_PACK() return the error code.
+ERR_PACK() return the error code.
 ERR_get_next_error_library() returns a new library number.
 
 =head1 SEE ALSO
diff --git a/lib/libssl/src/doc/crypto/RAND_cleanup.pod b/lib/libssl/src/doc/crypto/RAND_cleanup.pod
index 3a8f0749a8d..10cb39ce7ad 100644
--- a/lib/libssl/src/doc/crypto/RAND_cleanup.pod
+++ b/lib/libssl/src/doc/crypto/RAND_cleanup.pod
@@ -14,10 +14,6 @@ RAND_cleanup - erase the PRNG state
 
 RAND_cleanup() erases the memory used by the PRNG.
 
-=head1 RETURN VALUE
-
-RAND_cleanup() returns no value.
-
 =head1 SEE ALSO
 
 L<rand(3)|rand(3)>
diff --git a/lib/libssl/src/doc/crypto/RSA_blinding_on.pod b/lib/libssl/src/doc/crypto/RSA_blinding_on.pod
index 33990207f79..f96e3cf7c96 100644
--- a/lib/libssl/src/doc/crypto/RSA_blinding_on.pod
+++ b/lib/libssl/src/doc/crypto/RSA_blinding_on.pod
@@ -30,8 +30,6 @@ the blinding factor.
 
 RSA_blinding_on() returns 1 on success, and 0 if an error occurred.
 
-RSA_blinding_off() returns no value.
-
 =head1 SEE ALSO
 
 L<rsa(3)|rsa(3)>, L<rand(3)|rand(3)>
diff --git a/lib/libssl/src/doc/crypto/RSA_new.pod b/lib/libssl/src/doc/crypto/RSA_new.pod
index 41e5e60340d..0c85dc1d625 100644
--- a/lib/libssl/src/doc/crypto/RSA_new.pod
+++ b/lib/libssl/src/doc/crypto/RSA_new.pod
@@ -26,8 +26,6 @@ If the allocation fails, RSA_new() returns B<NULL> and sets an error code that
 can be obtained by L<ERR_get_error(3)|ERR_get_error(3)>. Otherwise it returns a
 pointer to the newly allocated structure.
 
-RSA_free() returns no value.
-
 =head1 SEE ALSO
 
 L<ERR_get_error(3)|ERR_get_error(3)>, L<rsa(3)|rsa(3)>,
diff --git a/lib/libssl/src/doc/crypto/RSA_set_method.pod b/lib/libssl/src/doc/crypto/RSA_set_method.pod
index 7f687c87189..3f50a89e5c4 100644
--- a/lib/libssl/src/doc/crypto/RSA_set_method.pod
+++ b/lib/libssl/src/doc/crypto/RSA_set_method.pod
@@ -142,8 +142,6 @@ the default method is used.
 RSA_PKCS1_SSLeay(), RSA_PKCS1_null_method(), RSA_get_default_method()
 and RSA_get_method() return pointers to the respective RSA_METHODs.
 
-RSA_set_default_method() returns no value.
-
 RSA_set_method() returns a pointer to the old RSA_METHOD implementation
 that was replaced. However, this return value should probably be ignored
 because if it was supplied by an ENGINE, the pointer could be invalidated
diff --git a/lib/libssl/src/doc/crypto/X509_new.pod b/lib/libssl/src/doc/crypto/X509_new.pod
index fd5fc65ce15..a36808b825c 100644
--- a/lib/libssl/src/doc/crypto/X509_new.pod
+++ b/lib/libssl/src/doc/crypto/X509_new.pod
@@ -6,6 +6,8 @@ X509_new, X509_free - X509 certificate ASN1 allocation functions
 
 =head1 SYNOPSIS
 
+ #include <openssl/x509.h>
+
  X509 *X509_new(void);
  void X509_free(X509 *a);
 
@@ -17,6 +19,7 @@ X509 structure, which represents an X509 certificate.
 X509_new() allocates and initializes a X509 structure.
 
 X509_free() frees up the B<X509> structure B<a>.
+If B<a> is a B<NULL> pointer, no action occurs.
 
 =head1 RETURN VALUES
 
@@ -24,8 +27,6 @@ If the allocation fails, X509_new() returns B<NULL> and sets an error
 code that can be obtained by L<ERR_get_error(3)|ERR_get_error(3)>.
 Otherwise it returns a pointer to the newly allocated structure.
 
-X509_free() returns no value.
-
 =head1 SEE ALSO
 
 L<ERR_get_error(3)|ERR_get_error(3)>, L<d2i_X509(3)|d2i_X509(3)>