[jira] [Commented] (WAGON-485) ScpWagon parses file size to int causing overflow for large files

2017-01-09 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/WAGON-485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15812972#comment-15812972
 ] 

Hudson commented on WAGON-485:
--

SUCCESS: Integrated in Jenkins build maven-wagon #1334 (See 
[https://builds.apache.org/job/maven-wagon/1334/])
[WAGON-485] ScpWagon parses file size to int causing overflow for large 
(michaelo: 
[http://git-wip-us.apache.org/repos/asf/?p=maven-wagon.git=commit=3512a321ef678209167fa978d5aafe2fdf6bfff7])
* (edit) 
wagon-providers/wagon-ssh/src/main/java/org/apache/maven/wagon/providers/ssh/jsch/ScpWagon.java


> ScpWagon parses file size to int causing overflow for large files
> -
>
> Key: WAGON-485
> URL: https://issues.apache.org/jira/browse/WAGON-485
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-ssh
>Affects Versions: 2.10
>Reporter: Silvio Assuncao
>Assignee: Michael Osipov
>Priority: Critical
> Fix For: 2.12
>
>
> {{ScpWagon}} class parses a string value to integer (file size) but if you 
> have a large file (3.6 GB for example) the file size is 3865470566 in bytes, 
> the number is larger than the {{MAX_VALUE}} of integer.
> {code:title=ScpWagon.java|borderStyle=solid}
> public void fillInputData( InputData inputData )
> throws TransferFailedException, ResourceDoesNotExistException {
> ...
> int filesize = Integer.valueOf( line.substring( 5, index ) ).intValue();
> fireTransferDebug( "Remote file size: " + filesize );
> resource.setContentLength( filesize );
> ...
> {code}
> I changed the line to parse using {{Long}} because the 
> *resource.setContentLength* method receive a long value too.
> {code:title=ScpWagon.java|borderStyle=solid}
> long filesize = Long.valueOf( line.substring( 5, index ) );
> {code}
> Please check the pull-request https://github.com/apache/maven-wagon/pull/32



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (WAGON-485) ScpWagon parses file size to int causing overflow for large files

2017-01-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/WAGON-485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15812920#comment-15812920
 ] 

ASF GitHub Bot commented on WAGON-485:
--

Github user asfgit closed the pull request at:

https://github.com/apache/maven-wagon/pull/32


> ScpWagon parses file size to int causing overflow for large files
> -
>
> Key: WAGON-485
> URL: https://issues.apache.org/jira/browse/WAGON-485
> Project: Maven Wagon
>  Issue Type: Bug
>  Components: wagon-ssh
>Affects Versions: 2.10
>Reporter: Silvio Assuncao
>Assignee: Michael Osipov
>Priority: Critical
> Fix For: 2.12
>
>
> {{ScpWagon}} class parses a string value to integer (file size) but if you 
> have a large file (3.6 GB for example) the file size is 3865470566 in bytes, 
> the number is larger than the {{MAX_VALUE}} of integer.
> {code:title=ScpWagon.java|borderStyle=solid}
> public void fillInputData( InputData inputData )
> throws TransferFailedException, ResourceDoesNotExistException {
> ...
> int filesize = Integer.valueOf( line.substring( 5, index ) ).intValue();
> fireTransferDebug( "Remote file size: " + filesize );
> resource.setContentLength( filesize );
> ...
> {code}
> I changed the line to parse using {{Long}} because the 
> *resource.setContentLength* method receive a long value too.
> {code:title=ScpWagon.java|borderStyle=solid}
> long filesize = Long.valueOf( line.substring( 5, index ) );
> {code}
> Please check the pull-request https://github.com/apache/maven-wagon/pull/32



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)