Index: src/main/java/org/apache/maven/plugin/javadoc/AbstractFixJavadocMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugin/javadoc/AbstractFixJavadocMojo.java	(revision 805744)
+++ src/main/java/org/apache/maven/plugin/javadoc/AbstractFixJavadocMojo.java	(working copy)
@@ -416,7 +416,8 @@
      */
     protected List getProjectSourceRoots( MavenProject p )
     {
-        return p.getCompileSourceRoots();
+        return ( p.getCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getCompileSourceRoots() ) );
     }
 
     /**
@@ -427,7 +428,8 @@
     protected List getCompileClasspathElements( MavenProject p )
         throws DependencyResolutionRequiredException
     {
-        return p.getCompileClasspathElements();
+        return ( p.getCompileClasspathElements() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getCompileClasspathElements() ) );
     }
 
     /**
Index: src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java	(revision 805778)
+++ src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java	(working copy)
@@ -35,6 +35,7 @@
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -1568,7 +1569,8 @@
             return Collections.EMPTY_LIST;
         }
 
-        return p.getCompileSourceRoots();
+        return ( p.getCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getCompileSourceRoots() ) );
     }
 
     /**
@@ -1582,7 +1584,8 @@
             return Collections.EMPTY_LIST;
         }
 
-        return p.getExecutionProject().getCompileSourceRoots();
+        return ( p.getExecutionProject().getCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getExecutionProject().getCompileSourceRoots() ) );
     }
 
     /**
@@ -1591,7 +1594,8 @@
      */
     protected List getProjectArtifacts( MavenProject p )
     {
-        return p.getCompileArtifacts();
+        return ( p.getCompileArtifacts() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getCompileArtifacts() ) );
     }
 
     /**
Index: src/main/java/org/apache/maven/plugin/javadoc/TestFixJavadocMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugin/javadoc/TestFixJavadocMojo.java	(revision 803855)
+++ src/main/java/org/apache/maven/plugin/javadoc/TestFixJavadocMojo.java	(working copy)
@@ -19,6 +19,8 @@
  * under the License.
  */
 
+import java.util.Collections;
+import java.util.LinkedList;
 import java.util.List;
 
 import org.apache.maven.artifact.DependencyResolutionRequiredException;
@@ -43,14 +45,16 @@
     /** {@inheritDoc} */
     protected List getProjectSourceRoots( MavenProject p )
     {
-        return p.getTestCompileSourceRoots();
+        return ( p.getTestCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getTestCompileSourceRoots() ) );
     }
 
     /** {@inheritDoc} */
     protected List getCompileClasspathElements( MavenProject p )
         throws DependencyResolutionRequiredException
     {
-        return p.getTestClasspathElements();
+        return ( p.getTestClasspathElements() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getTestClasspathElements() ) );
     }
 
     /** {@inheritDoc} */
Index: src/main/java/org/apache/maven/plugin/javadoc/TestJavadocReport.java
===================================================================
--- src/main/java/org/apache/maven/plugin/javadoc/TestJavadocReport.java	(revision 803855)
+++ src/main/java/org/apache/maven/plugin/javadoc/TestJavadocReport.java	(working copy)
@@ -22,6 +22,7 @@
 import java.io.File;
 import java.util.ArrayList;
 import java.util.Collections;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.ResourceBundle;
@@ -246,7 +247,8 @@
             return Collections.EMPTY_LIST;
         }
 
-        return p.getTestCompileSourceRoots();
+        return ( p.getTestCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getTestCompileSourceRoots() ) );
     }
 
     /** {@inheritDoc} */
@@ -257,13 +259,14 @@
             return Collections.EMPTY_LIST;
         }
 
-        return p.getExecutionProject().getTestCompileSourceRoots();
+        return ( p.getExecutionProject().getTestCompileSourceRoots() == null ? Collections.EMPTY_LIST
+                        : new LinkedList( p.getExecutionProject().getTestCompileSourceRoots() ) );
     }
 
     /** {@inheritDoc} */
     protected List getProjectArtifacts( MavenProject p )
     {
-        return p.getTestArtifacts();
+        return ( p.getTestArtifacts() == null ? Collections.EMPTY_LIST : new LinkedList( p.getTestArtifacts() ) );
     }
 
     /** {@inheritDoc} */