opendev/gerrit
Code Issues Proposed changes

Merge "Fix possible NPE in PatchDetailServiceImpl"

Browse Source
This commit is contained in:
Shawn Pearce
2013-08-22 15:49:34 +00:00
committed by Gerrit Code Review
parent 0958064aa0 7a940a787c
commit e8663e91fc
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java
View File

@@ -139,7 +139,7 @@ class PatchDetailServiceImpl extends BaseServiceImplementation implements
} }
return changeDetailFactory.create(result.getChangeId()).call(); return changeDetailFactory.create(result.getChangeId()).call();
} catch (NoSuchChangeException e) { } catch (NoSuchChangeException e) {
throw new Failure(new NoSuchChangeException(result.getChangeId())); throw new Failure(new NoSuchChangeException(psid.getParentKey()));
} catch (NoSuchProjectException e) { } catch (NoSuchProjectException e) {
throw new Failure(e); throw new Failure(e);
} catch (NoSuchEntityException e) { } catch (NoSuchEntityException e) {