Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Autoboxing NullPointerException #956

Closed
sampov2 opened this issue Aug 5, 2015 · 0 comments
Closed

Autoboxing NullPointerException #956

sampov2 opened this issue Aug 5, 2015 · 0 comments
Assignees
@andrey-qlogic
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. 🚨 This issue needs some love. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@sampov2
Copy link

sampov2 commented Aug 5, 2015

The MediaHttpDownloader in simple mode has a potential nullpointer issue because of autoboxing. response.getHeaders().getContentLength() returns a Long, which can be null in some cases. The MediaHttpDownloader.download() method stores this Long in a long field without checking for nulls.
@mattwhisenhunt mattwhisenhunt added status: investigating type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Dec 22, 2017
netp-dany added a commit to netp-dany/google-api-java-client that referenced this issue May 29, 2018
@netp-dany
fix for Autoboxing NullPointerException googleapis#956
ae52739
netp-dany added a commit to netp-dany/google-api-java-client that referenced this issue May 29, 2018
@netp-dany
Revert "fix for Autoboxing NullPointerException googleapis#956"
493d666 
This reverts commit ae52739
@JustinBeckwith JustinBeckwith added priority: p2 Moderately-important priority. Fix may not be included in next release. 🚨 This issue needs some love. and removed status: investigating labels Jun 7, 2018
@andrey-qlogic andrey-qlogic mentioned this issue Feb 13, 2019
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrey-qlogic andrey-qlogic

Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. 🚨 This issue needs some love. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@JustinBeckwith @sampov2 @mattwhisenhunt @andrey-qlogic