[ https://issues.apache.org/jira/browse/FLINK-4309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15422393#comment-15422393 ]
ASF GitHub Bot commented on FLINK-4309: --------------------------------------- Github user uce commented on a diff in the pull request: https://github.com/apache/flink/pull/2371#discussion_r74893448 --- Diff: flink-core/src/main/java/org/apache/flink/configuration/DelegatingConfiguration.java --- @@ -178,14 +178,19 @@ public String toString() { @Override public Set<String> keySet() { + --- End diff -- We usually don't have an empty line when a method block starts (also applies to lines 185 and in `DelegatingConfigurationTest` lines 96 and 110). > Potential null pointer dereference in DelegatingConfiguration#keySet() > ---------------------------------------------------------------------- > > Key: FLINK-4309 > URL: https://issues.apache.org/jira/browse/FLINK-4309 > Project: Flink > Issue Type: Bug > Reporter: Ted Yu > Assignee: Sunny T > Priority: Minor > > {code} > final int prefixLen = this.prefix == null ? 0 : this.prefix.length(); > for (String key : this.backingConfig.keySet()) { > if (key.startsWith(this.prefix)) { > {code} > If this.prefix == null, we would get NPE in startsWith(): > {code} > public boolean startsWith(String prefix, int toffset) { > char ta[] = value; > int to = toffset; > char pa[] = prefix.value; > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)