diff --git a/jdk/src/share/classes/java/security/AccessController.java b/jdk/src/share/classes/java/security/AccessController.java
index ed103a9186d..a7d089958fe 100644
--- a/jdk/src/share/classes/java/security/AccessController.java
+++ b/jdk/src/share/classes/java/security/AccessController.java
@@ -279,6 +279,9 @@ public final class AccessController {
*
Note that any DomainCombiner associated with the current
* AccessControlContext will be ignored while the action is performed.
*
+ * @param the type of the value returned by the PrivilegedAction's
+ * {@code run} method.
+ *
* @param action the action to be performed.
*
* @return the value returned by the action's {@code run} method.
@@ -305,6 +308,9 @@ public final class AccessController {
*
This method preserves the current AccessControlContext's
* DomainCombiner (which may be null) while the action is performed.
*
+ * @param the type of the value returned by the PrivilegedAction's
+ * {@code run} method.
+ *
* @param action the action to be performed.
*
* @return the value returned by the action's {@code run} method.
@@ -344,6 +350,8 @@ public final class AccessController {
* {@link java.security.SecurityPermission}, then the action is performed
* with no permissions.
*
+ * @param the type of the value returned by the PrivilegedAction's
+ * {@code run} method.
* @param action the action to be performed.
* @param context an access control context
* representing the restriction to be applied to the
@@ -377,6 +385,8 @@ public final class AccessController {
* If the action's {@code run} method throws an (unchecked) exception,
* it will propagate through this method.
*
+ * @param the type of the value returned by the PrivilegedAction's
+ * {@code run} method.
* @param action the action to be performed.
* @param context an access control context
* representing the restriction to be applied to the
@@ -429,6 +439,8 @@ public final class AccessController {
*
This method preserves the current AccessControlContext's
* DomainCombiner (which may be null) while the action is performed.
*
+ * @param the type of the value returned by the PrivilegedAction's
+ * {@code run} method.
* @param action the action to be performed.
* @param context an access control context
* representing the restriction to be applied to the
@@ -479,6 +491,9 @@ public final class AccessController {
*
Note that any DomainCombiner associated with the current
* AccessControlContext will be ignored while the action is performed.
*
+ * @param the type of the value returned by the
+ * PrivilegedExceptionAction's {@code run} method.
+ *
* @param action the action to be performed
*
* @return the value returned by the action's {@code run} method
@@ -509,6 +524,9 @@ public final class AccessController {
*
This method preserves the current AccessControlContext's
* DomainCombiner (which may be null) while the action is performed.
*
+ * @param the type of the value returned by the
+ * PrivilegedExceptionAction's {@code run} method.
+ *
* @param action the action to be performed.
*
* @return the value returned by the action's {@code run} method
@@ -585,6 +603,8 @@ public final class AccessController {
* {@link java.security.SecurityPermission}, then the action is performed
* with no permissions.
*
+ * @param the type of the value returned by the
+ * PrivilegedExceptionAction's {@code run} method.
* @param action the action to be performed
* @param context an access control context
* representing the restriction to be applied to the
@@ -622,6 +642,8 @@ public final class AccessController {
* If the action's {@code run} method throws an (unchecked) exception,
* it will propagate through this method.
*
+ * @param the type of the value returned by the
+ * PrivilegedExceptionAction's {@code run} method.
* @param action the action to be performed.
* @param context an access control context
* representing the restriction to be applied to the
@@ -676,6 +698,8 @@ public final class AccessController {
*
This method preserves the current AccessControlContext's
* DomainCombiner (which may be null) while the action is performed.
*
+ * @param the type of the value returned by the
+ * PrivilegedExceptionAction's {@code run} method.
* @param action the action to be performed.
* @param context an access control context
* representing the restriction to be applied to the
diff --git a/jdk/src/share/classes/java/security/AlgorithmParameters.java b/jdk/src/share/classes/java/security/AlgorithmParameters.java
index c603a196c9f..b548fcb64c8 100644
--- a/jdk/src/share/classes/java/security/AlgorithmParameters.java
+++ b/jdk/src/share/classes/java/security/AlgorithmParameters.java
@@ -324,6 +324,7 @@ public class AlgorithmParameters {
* parameters should be returned in an instance of the
* {@code DSAParameterSpec} class.
*
+ * @param the type of the parameter specification to be returrned
* @param paramSpec the specification class in which
* the parameters should be returned.
*
diff --git a/jdk/src/share/classes/java/security/AlgorithmParametersSpi.java b/jdk/src/share/classes/java/security/AlgorithmParametersSpi.java
index be231a4cafe..282493b97b9 100644
--- a/jdk/src/share/classes/java/security/AlgorithmParametersSpi.java
+++ b/jdk/src/share/classes/java/security/AlgorithmParametersSpi.java
@@ -102,6 +102,8 @@ public abstract class AlgorithmParametersSpi {
* parameters should be returned in an instance of the
* {@code DSAParameterSpec} class.
*
+ * @param the type of the parameter specification to be returned
+ *
* @param paramSpec the specification class in which
* the parameters should be returned.
*
diff --git a/jdk/src/share/classes/java/security/KeyFactory.java b/jdk/src/share/classes/java/security/KeyFactory.java
index 0eb6b754107..8e761ff41f7 100644
--- a/jdk/src/share/classes/java/security/KeyFactory.java
+++ b/jdk/src/share/classes/java/security/KeyFactory.java
@@ -395,6 +395,8 @@ public class KeyFactory {
* key material should be returned in an instance of the
* {@code DSAPublicKeySpec} class.
*
+ * @param the type of the key specification to be returned
+ *
* @param key the key.
*
* @param keySpec the specification class in which
diff --git a/jdk/src/share/classes/java/security/KeyFactorySpi.java b/jdk/src/share/classes/java/security/KeyFactorySpi.java
index 877c3a11be1..5ee7f458931 100644
--- a/jdk/src/share/classes/java/security/KeyFactorySpi.java
+++ b/jdk/src/share/classes/java/security/KeyFactorySpi.java
@@ -106,6 +106,8 @@ public abstract class KeyFactorySpi {
* key material should be returned in an instance of the
* {@code DSAPublicKeySpec} class.
*
+ * @param the type of the key specification to be returned
+ *
* @param key the key.
*
* @param keySpec the specification class in which
diff --git a/jdk/src/share/classes/java/security/KeyStore.java b/jdk/src/share/classes/java/security/KeyStore.java
index c363d0719f7..187683baa50 100644
--- a/jdk/src/share/classes/java/security/KeyStore.java
+++ b/jdk/src/share/classes/java/security/KeyStore.java
@@ -1753,6 +1753,7 @@ public class KeyStore {
/**
* Returns the KeyStore described by this object.
*
+ * @return the {@code KeyStore} described by this object
* @exception KeyStoreException if an error occured during the
* operation, for example if the KeyStore could not be
* instantiated or loaded
diff --git a/jdk/src/share/classes/java/security/Principal.java b/jdk/src/share/classes/java/security/Principal.java
index 48938cfdd07..a538e707ee7 100644
--- a/jdk/src/share/classes/java/security/Principal.java
+++ b/jdk/src/share/classes/java/security/Principal.java
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 1996, 2012, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1996, 2013, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -81,6 +81,7 @@ public interface Principal {
*
Subclasses may override this with a different implementation, if
* necessary.
*
+ * @param subject the {@code Subject}
* @return true if {@code subject} is non-null and is
* implied by this principal, or false otherwise.
* @since 1.8
diff --git a/jdk/src/share/classes/java/security/cert/CertPathBuilderSpi.java b/jdk/src/share/classes/java/security/cert/CertPathBuilderSpi.java
index 87908c03bd9..e7755411797 100644
--- a/jdk/src/share/classes/java/security/cert/CertPathBuilderSpi.java
+++ b/jdk/src/share/classes/java/security/cert/CertPathBuilderSpi.java
@@ -87,6 +87,8 @@ public abstract class CertPathBuilderSpi {
* service providers, this method cannot be abstract and by default throws
* an {@code UnsupportedOperationException}.
*
+ * @return a {@code CertPathChecker} that this implementation uses to
+ * check the revocation status of certificates
* @throws UnsupportedOperationException if this method is not supported
* @since 1.8
*/
diff --git a/jdk/src/share/classes/java/security/cert/CertPathValidatorSpi.java b/jdk/src/share/classes/java/security/cert/CertPathValidatorSpi.java
index 50ad9c85c9b..02d503c9e62 100644
--- a/jdk/src/share/classes/java/security/cert/CertPathValidatorSpi.java
+++ b/jdk/src/share/classes/java/security/cert/CertPathValidatorSpi.java
@@ -97,6 +97,8 @@ public abstract class CertPathValidatorSpi {
* service providers, this method cannot be abstract and by default throws
* an {@code UnsupportedOperationException}.
*
+ * @return a {@code CertPathChecker} that this implementation uses to
+ * check the revocation status of certificates
* @throws UnsupportedOperationException if this method is not supported
* @since 1.8
*/
diff --git a/jdk/src/share/classes/java/security/cert/PKIXRevocationChecker.java b/jdk/src/share/classes/java/security/cert/PKIXRevocationChecker.java
index b667397c8e5..3046a03ed23 100644
--- a/jdk/src/share/classes/java/security/cert/PKIXRevocationChecker.java
+++ b/jdk/src/share/classes/java/security/cert/PKIXRevocationChecker.java
@@ -103,6 +103,9 @@ public abstract class PKIXRevocationChecker extends PKIXCertPathChecker {
private Map ocspResponses = Collections.emptyMap();
private Set