Heya, I’m looking at fixing a potential NPE in the PrimaryStorageDownloadCommand. When the constructor 'public PrimaryStorageDownloadCommand(String url, StoragePool pool, int wait) ‘ is used it calls 'super(null, url, null, null);’ which in the constructor unboxes the fourth parameter from a Long to a long. This would generate an NPE.
There is several easy fixes, but looking through the code i failed to find where this command is used. Can somebody shine a light on this, or is this class a relic from days gone by that we could remove. CC Edison as he last touched this piece of code Cheers, Hugo
