----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22726 -----------------------------------------------------------
Commit 6b3257781173f11113596114c5b19366e72ffb46 in branch refs/heads/master-6-17-stable from Sanjay Tripathi [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6b32577 ] CLOUDSTACK-3184: updateVirtualMachine api gives java NPE. Signed-off-by: Prasanna Santhanam <t...@apache.org> (cherry picked from commit 31df729008c748e6eafe49ff86e1da618013c551) - ASF Subversion and Git Services On July 3, 2013, 11:21 a.m., Sanjay Tripathi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12243/ > ----------------------------------------------------------- > > (Updated July 3, 2013, 11:21 a.m.) > > > Review request for cloudstack, Devdeep Singh and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-3184 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-3184: updateVirtualMachine api gives java NPE. > > This bug is not reproducible from CloudStack UI. Steps to reproduce this bug: > > 1. Deploy a VM without passing any parameters in Name and Description (last > step in deploy VM wizard). > 2. Stop this VM. > 3. Fire updateVIrtualMachine API without passing "displayName" parameter, > you'll see NPE in MS logs. Do not try with UI because UI will pass > displayName paramter with blank value. > > There isn't any null check for displayName parameter and it is null because > initially it was not set during the VM deployment and afterwards we are not > passing displayName paramter in the API. > > So added null check for displayName in if condition. > > > Diffs > ----- > > server/src/com/cloud/vm/UserVmManagerImpl.java a59fa5b > > Diff: https://reviews.apache.org/r/12243/diff/ > > > Testing > ------- > > Same as Steps mentioned in the description. > > > Thanks, > > Sanjay Tripathi > >
