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.

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

NullPointerException #2879

Closed
GavinZhoug opened this issue Feb 7, 2018 · 3 comments
Closed

NullPointerException #2879

GavinZhoug opened this issue Feb 7, 2018 · 3 comments
Labels

Comments

@GavinZhoug
Copy link

GavinZhoug commented Feb 7, 2018

Issue happened when I scroll UI fastly. How do I fix this bug? or avoid it. thank you.
Glide verson 4.6.1
Andoird verson 6.0
Integration libraries: glide-4.6.1-full.jar; annotation-4.6.1.jar;
Glide load Line:
Glide.with(context).load(url).transition(withCrossFade()).into(view);

E/AndroidRuntime( 4263): FATAL EXCEPTION: Thread-171
E/AndroidRuntime( 4263): Process: com.mgtv.tv.desktop, PID: 4263
E/AndroidRuntime( 4263): java.lang.NullPointerException: Argument must not be null
E/AndroidRuntime( 4263): at
com.bumptech.glide.util.Preconditions.checkNotNull(Preconditions.java:31)
E/AndroidRuntime( 4263): at
com.bumptech.glide.util.Preconditions.checkNotNull(Preconditions.java:25)
E/AndroidRuntime( 4263): at com.bumptech.glide.load.model.MultiModelLoader$MultiFetcher.onLoadFailed(MultiModelLoader.java:141)
E/AndroidRuntime( 4263): at com.bumptech.glide.integration.volley.VolleyStreamFetcher$GlideRequest.parseNetworkError(VolleyStreamFetcher.java:136)
E/AndroidRuntime( 4263): at com.android.volley.NetworkDispatcher.parseAndDeliverNetworkError(NetworkDis
_20180207143520
patcher.java:149)
E/AndroidRuntime( 4263): at com.android.volley.NetworkDispatcher.run(NetworkDispatcher.java:138)

@sjudd
Copy link
Collaborator

sjudd commented Feb 8, 2018

Hi @GavinZhoug,

It looks like you might not have included some important information when you filed this issue. To make sure we have all the information we need to help with your issue. Please take a look at the Issue Template and make sure you've filled out as many sections as possible.

In particular you must include:

  • Glide Version
  • Glide load line
  • Issue details.

I'm marking this issue as requiring more information. Please understand that if you do not provide that information or update the issue within the next 7 days this issue will be automatically closed so that it doesn''t clutter the bug tracker.

Best regards,

~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

@GavinZhoug
Copy link
Author

Glide verson 4.6.1
Andoird verson 6.0
Integration libraries: glide-4.6.1-full.jar; annotation-4.6.1.jar;
Glide load Line:
Glide.with(context).load(url).transition(withCrossFade()).into(view);

@sjudd
Copy link
Collaborator

sjudd commented Feb 8, 2018

I believe this happens because the volley integration library may notify that a load fails after we've cancelled it. I have a fix for that, but I'm not sure there's much you're going to be able to do to work around the issue other than use a different networking library.

@sjudd sjudd closed this as completed in a690f38 Feb 9, 2018
sjudd added a commit to sjudd/glide that referenced this issue Feb 21, 2018
…dded to fix bumptech#2879

This wasn't well throught through. Avoiding calling callbacks can lead
to resource leaks.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=185931910
sjudd added a commit to sjudd/glide that referenced this issue Mar 6, 2018
-------------
Include the debug aar in release artifacts for Android projects.

We removed the release variant a while ago to speed up the build, which
has the side affect of removing the release aar from artifacts. Since
we expect the debug and release variants to be identical (hence why
we disabled the release variant), it should be safe to just use the
debug aar instead. We will have to specify it explicitly since android’s
rules unsurprisingly only add the release variant by default.

-------------
Bump version to 4.6.0

-------------
Update readme to 4.6.0

Also removes the old v4 dependency from maven deps, I don’t think it’s
necessary.

-------------
Change update_javadocs to use debugJavadocJar instead of release.

We’ve disabled the release variant.

-------------
Bump version to 4.7.0-SNAPSHOT

-------------
Add POM dependencies explicitly.

Fixes bumptech#2863.

-------------
Bump version to 4.6.1

-------------
Update readme to 4.6.1

-------------
Fix param mistake (bumptech#2873)

-------------
Update SimpleTarget javadoc to match v4 API.

-------------
Add javadoc for RequestOptions.apply/RequestBuilder.apply.

Related to bumptech#2858.

-------------
Add support for Uri data uris.

Previously we only supported data uris if they were provided as Strings.

Fixes bumptech#556.

-------------
Make GlideBuilder.build package private.

It shouldn’t have been made visible and can’t
safely be used directly outside of the library.

Fixes bumptech#2885

-------------
Handle notifications in MultiFetcher after cancellation.

We can’t guarantee that every fetcher implementation will strictly avoid
notifying after they’ve been cancelled.

This also improves the behavior in VolleyStreamFetcher so that it attempts to avoid notifying after cancel, although it still doesn’t make
any strict guarantee.

Fixes bumptech#2879

-------------
Re-enable -Werror for java compilation.

Related to bumptech#2886.

-------------
Fix a deprecation warning in DataUriTest.

-------------
gradle 4.5.1

-------------
deprecate fragments

-------------
add @deprecated to javadoc and suppress deprecations in code

-------------
Created by MOE: https://github.com/google/moe

MOE_MIGRATED_REVID=99229725401d5777e059da7b6331134bf73fbcdf

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=185535564
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants