RFC: Patch to fix rhbz# 560193

[Deepak Bhole]

* Deepak Bhole <dbhole at redhat.com> [2010-08-27 15:32]:
> Hi,
> 
> I hastily (and mistakenly) committed this patch to the branches before
> posting it here. Attaching it now. If there are any objections, I will
> back out the changes.
> 


Oops.. just saw Andrew's message saying it was clean. Nevermind this RFC
:)

Deepak

> ChangeLog:
> 
> 2010-08-27  Deepak Bhole <dbhole at redhat.com>
> 
>     Fixes rhbz#560193
>     * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
>     (activateJars): Process nested jar only if size > 0 bytes.
> 
> 
> Cheers,
> Deepak

> diff -r 627620efacc8 -r ead30a628023 netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
> --- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Thu Aug 26 11:03:17 2010 +0200
> +++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Fri Aug 27 13:59:35 2010 -0400
> @@ -638,13 +638,20 @@
>  
>                                      byte[] bytes = new byte[1024];
>                                      int read = is.read(bytes);
> +                                    int fileSize = read;
>                                      while (read > 0) {
>                                          extractedJar.write(bytes, 0, read);
>                                          read = is.read(bytes);
> +                                        fileSize += read;
>                                      }
>  
>                                      is.close();
>                                      extractedJar.close();
> +                                    
> +                                    // 0 byte file? skip
> +                                    if (fileSize <= 0) {
> +                                    	continue;
> +                                    }
>  
>                                      JarSigner signer = new JarSigner();
>                                      signer.verifyJar(extractedJarLocation);