Re: [PATCH 0/2] Fixed issues/defects reported by Coverity tool.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Please find attached zipped patches.


On Mon, 2010-12-06 at 13:51 +0100, ext Clemens Ladisch wrote:
> sudarshan.bisht@xxxxxxxxx wrote:
> > Sudarshan (1):
> >   alsa-lib: Removed trailing whitespaces as per pulseaudio coding
> >     guidelines.
> > 
> > bisht (1):
> >   alsa-lib: Coverity tool has picked some minor issues in alsa-lib
> >     source code, fixed them.
> 
> Apparently, your patches were too big and got stuck in some spam filter.
> 
> Try to compress them, or put them somewhere on the web.
> 
> 
> Regards,
> Clemens
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

>From 5c01f696c0cbcc9b59ca9af32dd42f8efcaa34a0 Mon Sep 17 00:00:00 2001
From: Sudarshan <sudarshan.bisht@xxxxxxxxx>
Date: Fri, 3 Dec 2010 13:59:10 +0200
Subject: [PATCH 0/2] Fixed issues/defects reported by Coverity tool.

	
	Coverity Static Analysis helps developers find hard-to-spot, 
yet potentially crash-causing defects early in the development phase,
reducing the cost,time, and risk of software errors.


* First patch here is just a cosmetic one as it removes 
tabs and trailing whitespaces from the code.

* Second patch has got fixes for some minor issues/defects 
reported by Coverity tool and these fixes can prevent memory 
leaks and crashes in some cases. It also has some whitespaces 
and tabs removal differences, please ignore them.

Sudarshan (1):
  alsa-lib: Removed trailing whitespaces as per pulseaudio coding
    guidelines.

bisht (1):
  alsa-lib: Coverity tool has picked some minor issues in alsa-lib
    source code, fixed them.

 aserver/aserver.c              | 1813 +++++++-------
 modules/mixer/simple/sbase.c   |  837 ++++----
 modules/mixer/simple/sbasedl.c |  126 +-
 src/alisp/alisp.c              |    4 +-
 src/alisp/alisp_snd.c          | 1493 ++++++------
 src/conf.c                     | 5117 ++++++++++++++++++++--------------------
 src/control/control_hw.c       |  597 +++---
 src/control/control_shm.c      | 1019 ++++----
 src/control/setup.c            |    3 +-
 src/hwdep/hwdep.c              |  496 ++--
 src/mixer/mixer.c              |  962 ++++----
 src/mixer/simple_abst.c        |    1 +
 src/mixer/simple_none.c        |   15 +-
 src/pcm/pcm.c                  | 4812 +++++++++++++++++++-------------------
 src/pcm/pcm_file.c             | 1452 ++++++------
 src/pcm/pcm_hooks.c            |  937 ++++----
 src/pcm/pcm_ladspa.c           | 2284 +++++++++---------
 src/pcm/pcm_local.h            |    3 +-
 src/pcm/pcm_meter.c            | 1634 +++++++-------
 src/pcm/pcm_rate.c             | 2166 +++++++++---------
 src/pcm/pcm_share.c            |   10 +-
 src/pcm/pcm_shm.c              | 1329 ++++++------
 src/pcm/pcm_simple.c           |  404 ++--
 src/rawmidi/rawmidi.c          |    2 +-
 src/rawmidi/rawmidi_virt.c     |    8 +-
 25 files changed, 13782 insertions(+), 13742 deletions(-)

Attachment: gzSOYvGRsLbC.gz
Description: GNU Zip compressed data

Attachment: gz8qtBzXipok.gz
Description: GNU Zip compressed data

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

Add to Google Powered by Linux