{"action":"closed","issue":{"url":"https://api.github.com/repos/chrsmith/google-api-java-client/issues/254","labels_url":"https://api.github.com/repos/chrsmith/google-api-java-client/issues/254/labels{/name}","comments_url":"https://api.github.com/repos/chrsmith/google-api-java-client/issues/254/comments","events_url":"https://api.github.com/repos/chrsmith/google-api-java-client/issues/254/events","html_url":"https://github.com/chrsmith/google-api-java-client/issues/254","id":59373732,"number":254,"title":"FindBugs found: Dereferencing transport without confirming is not null","user":{"login":"GoogleCodeExporter","id":9614759,"avatar_url":"https://avatars.githubusercontent.com/u/9614759?v=3","gravatar_id":"","url":"https://api.github.com/users/GoogleCodeExporter","html_url":"https://github.com/GoogleCodeExporter","followers_url":"https://api.github.com/users/GoogleCodeExporter/followers","following_url":"https://api.github.com/users/GoogleCodeExporter/following{/other_user}","gists_url":"https://api.github.com/users/GoogleCodeExporter/gists{/gist_id}","starred_url":"https://api.github.com/users/GoogleCodeExporter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GoogleCodeExporter/subscriptions","organizations_url":"https://api.github.com/users/GoogleCodeExporter/orgs","repos_url":"https://api.github.com/users/GoogleCodeExporter/repos","events_url":"https://api.github.com/users/GoogleCodeExporter/events{/privacy}","received_events_url":"https://api.github.com/users/GoogleCodeExporter/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chrsmith/google-api-java-client/labels/auto-migrated","name":"auto-migrated","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/google-api-java-client/labels/Priority-Low","name":"Priority-Low","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/google-api-java-client/labels/Type-Defect","name":"Type-Defect","color":"ededed"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-01T03:10:17Z","updated_at":"2015-03-01T03:10:18Z","closed_at":"2015-03-01T03:10:17Z","body":"```\nIn the two places below, the transport field is dereferenced without checking \nthat it was initialized:\n\napi.client.googleapis.auth.clientlogin.ClientLogin\n\n  public Response authenticate() throws HttpResponseException, IOException {\n    GenericUrl url = serverUrl.clone();\n    url.appendRawPath(\"/accounts/ClientLogin\");\n    HttpRequest request =\n        transport.createRequestFactory().buildPostRequest(url, new UrlEncodedContent(this));\n    request.addParser(AuthKeyValueParser.INSTANCE);\n    request.setDisableContentLogging(true);\n    return request.execute().parseAs(Response.class);\n  }\n\napi.client.googleapis.json\n\n  private HttpRequest internalExecute(Object data) {\n    JsonHttpContent content = new JsonHttpContent(jsonFactory, data);\n    content.setType(contentType);\n    HttpRequest httpRequest;\n    try {\n      httpRequest = transport.createRequestFactory().buildPostRequest(rpcServerUrl, content);\n      httpRequest.getHeaders().setAccept(accept);\n      return httpRequest;\n    } catch (IOException e) {\n      throw new IllegalStateException(e);\n    }\n  }\n\n```\n\nOriginal issue reported on code.google.com by `avall...@google.com` on 28 Jul 2011 at 3:34\n* Merged into: #145"}}