Skip to content

Commit

Permalink
Avoid possible memory leak in ResolvableType
Browse files Browse the repository at this point in the history
  • Loading branch information
dreis2211 authored and jhoeller committed Apr 23, 2019
1 parent cd5dc84 commit 29d021a
Showing 1 changed file with 10 additions and 4 deletions.
Expand Up @@ -935,7 +935,7 @@ VariableResolver asVariableResolver() {
if (this == NONE) {
return null;
}
return new DefaultVariableResolver();
return new DefaultVariableResolver(this);
}

/**
Expand Down Expand Up @@ -1452,17 +1452,23 @@ interface VariableResolver extends Serializable {


@SuppressWarnings("serial")
private class DefaultVariableResolver implements VariableResolver {
private static class DefaultVariableResolver implements VariableResolver {

private final ResolvableType source;

DefaultVariableResolver(ResolvableType resolvableType) {
this.source = resolvableType;
}

@Override
@Nullable
public ResolvableType resolveVariable(TypeVariable<?> variable) {
return ResolvableType.this.resolveVariable(variable);
return this.source.resolveVariable(variable);
}

@Override
public Object getSource() {
return ResolvableType.this;
return this.source;
}
}

Expand Down

0 comments on commit 29d021a

Please sign in to comment.