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@

13 files changed, 4 insertions, 31 deletions

diff --git a/src/lib/libcrypto/doc/DH_new.pod b/src/lib/libcrypto/doc/DH_new.pod
index d6c3ca82b5..0fdb7b9680 100644
--- a/src/lib/libcrypto/doc/DH_new.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/DH_set_method.pod b/src/lib/libcrypto/doc/DH_set_method.pod
index 2a4c2735cb..d82fe7377a 100644
--- a/src/lib/libcrypto/doc/DH_set_method.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/DSA_SIG_new.pod b/src/lib/libcrypto/doc/DSA_SIG_new.pod
index 3ac6140038..77aa649db0 100644
--- a/src/lib/libcrypto/doc/DSA_SIG_new.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/DSA_new.pod b/src/lib/libcrypto/doc/DSA_new.pod
index 48e9b82a09..e1e30b9a07 100644
--- a/src/lib/libcrypto/doc/DSA_new.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/DSA_set_method.pod b/src/lib/libcrypto/doc/DSA_set_method.pod
index 31f444d751..bc57a3e8e2 100644
--- a/src/lib/libcrypto/doc/DSA_set_method.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/ERR_load_strings.pod b/src/lib/libcrypto/doc/ERR_load_strings.pod
index 5acdd0edbc..e9c5cf0fc5 100644
--- a/src/lib/libcrypto/doc/ERR_load_strings.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/RAND_cleanup.pod b/src/lib/libcrypto/doc/RAND_cleanup.pod
index 3a8f0749a8..10cb39ce7a 100644
--- a/src/lib/libcrypto/doc/RAND_cleanup.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/RSA_blinding_on.pod b/src/lib/libcrypto/doc/RSA_blinding_on.pod
index 33990207f7..f96e3cf7c9 100644
--- a/src/lib/libcrypto/doc/RSA_blinding_on.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/RSA_new.pod b/src/lib/libcrypto/doc/RSA_new.pod
index 41e5e60340..0c85dc1d62 100644
--- a/src/lib/libcrypto/doc/RSA_new.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/RSA_set_method.pod b/src/lib/libcrypto/doc/RSA_set_method.pod
index 7f687c8718..3f50a89e5c 100644
--- a/src/lib/libcrypto/doc/RSA_set_method.pod
+++ b/src/lib/libcrypto/doc/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/src/lib/libcrypto/doc/X509_new.pod b/src/lib/libcrypto/doc/X509_new.pod
index d38872335f..a36808b825 100644
--- a/src/lib/libcrypto/doc/X509_new.pod
+++ b/src/lib/libcrypto/doc/X509_new.pod
@@ -19,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
 
@@ -26,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)>
diff --git a/src/lib/libcrypto/man/ASN1_OBJECT_new.3 b/src/lib/libcrypto/man/ASN1_OBJECT_new.3
index 5b81b2bded..526e1d869b 100644
--- a/src/lib/libcrypto/man/ASN1_OBJECT_new.3
+++ b/src/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/src/lib/libcrypto/man/BUF_MEM_new.3 b/src/lib/libcrypto/man/BUF_MEM_new.3
index f87e4f8257..10258b7aaf 100644
--- a/src/lib/libcrypto/man/BUF_MEM_new.3
+++ b/src/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 ).