On Thu, Sep 03, 2009 at 12:49:25PM +0200, Jim Meyering wrote:
> Daniel P. Berrange wrote:
> > On Thu, Sep 03, 2009 at 11:18:00AM +0200, Jim Meyering wrote:
> >> clang was complaining that a NULL inputvol would be dereferenced
> >> in that "could not open..." diagnostic.
> >>
> >> Since the two sole
Daniel P. Berrange wrote:
> On Thu, Sep 03, 2009 at 11:18:00AM +0200, Jim Meyering wrote:
>> clang was complaining that a NULL inputvol would be dereferenced
>> in that "could not open..." diagnostic.
>>
>> Since the two sole callers of this function are careful
>> to call it only when inputvol is
Daniel P. Berrange wrote:
> On Thu, Sep 03, 2009 at 11:18:00AM +0200, Jim Meyering wrote:
>> clang was complaining that a NULL inputvol would be dereferenced
>> in that "could not open..." diagnostic.
>>
>> Since the two sole callers of this function are careful
>> to call it only when inputvol is
On Thu, Sep 03, 2009 at 11:18:00AM +0200, Jim Meyering wrote:
> clang was complaining that a NULL inputvol would be dereferenced
> in that "could not open..." diagnostic.
>
> Since the two sole callers of this function are careful
> to call it only when inputvol is non-NULL, this is a good
> case
clang was complaining that a NULL inputvol would be dereferenced
in that "could not open..." diagnostic.
Since the two sole callers of this function are careful
to call it only when inputvol is non-NULL, this is a good
case for giving the parameter the nonnull attribute:
>From 314278acb021a1f2e6