qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DISPATCH-954) Fix Coverity issue on master branch
Date Mon, 02 Apr 2018 18:05:00 GMT

    [ https://issues.apache.org/jira/browse/DISPATCH-954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16422889#comment-16422889
] 

ASF GitHub Bot commented on DISPATCH-954:
-----------------------------------------

GitHub user fgiorgetti opened a pull request:

    https://github.com/apache/qpid-dispatch/pull/276

    DISPATCH-954 - Fixed issue suggested by coverity scan tool

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/fgiorgetti/qpid-dispatch fgiorgetti-DISPATCH-954

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/qpid-dispatch/pull/276.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #276
    
----
commit 710b6da05d8d8e2e4f0b8fbbed18c7947c4949e9
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-03-08T17:51:52Z

    DISPATCH-940 - Allow configuration file to be used from relative path when running as
a daemon

commit a0b280a207027dd0b27469b59b93cbbe3ff5adfa
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-03-09T05:26:36Z

    DISPATCH-940 - Created unit test to validate command line parsing

commit 43d21a0df4c564d9eed6796e766149031d0feff3
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-03-09T12:49:56Z

    DISPATCH-940 - Improved code quality using pylint recommendations

commit 5afc09a26ef0a73ca897ca8dd6ad8b5e8bfd5b7b
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-03-09T16:53:40Z

    DISPATCH-940 - Implemented recommended coding standards

commit b202509425838643b9c7f0f553521210a5a484e8
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-04-02T16:40:55Z

    Merge branch 'master' of github.com:apache/qpid-dispatch
    
    Conflicts:
    	router/src/main.c
    	tests/CMakeLists.txt
    	tests/system_tests_cmdline_parsing.py

commit f09a8ee8e923d1ad56fd3a8679ac575e5c7092b1
Author: Fernando Giorgetti <fgiorget@...>
Date:   2018-04-02T16:45:54Z

    DISPATCH-954 - Fixed issue suggested by coverity scan tool

----


> Fix Coverity issue on master branch
> -----------------------------------
>
>                 Key: DISPATCH-954
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-954
>             Project: Qpid Dispatch
>          Issue Type: Bug
>          Components: Container
>    Affects Versions: 1.0.1
>            Reporter: Ganesh Murthy
>            Priority: Major
>             Fix For: 1.1.0
>
>
> Please find the latest report on new defect(s) introduced to Apache Qpid dispatch-router
found with Coverity Scan.
>  
>  1 new defect(s) introduced to Apache Qpid dispatch-router found with Coverity Scan.
>  4 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build
analyzed by Coverity Scan.
>  
>  New defect(s) Reported-by: Coverity Scan
>  Showing 1 of 1 defect(s)
>  
>  
>  ** CID 267696:  Memory - illegal accesses  (USE_AFTER_FREE)
>  /home/kgiusti/work/dispatch/qpid-dispatch/router/src/main.c: 215 in daemon_process()
>  
>  
>  ________________________________________________________________________________________________________
>  *** CID 267696:  Memory - illegal accesses  (USE_AFTER_FREE)
>  /home/kgiusti/work/dispatch/qpid-dispatch/router/src/main.c: 215 in daemon_process()
>  209                                  cur_path,
>  210                                  !strcmp("/", cur_path)? "":"/",
>  211                                  config_path);
>  212                     }
>  213
>  214                     // Releasing temporary path variable
>  >>>     CID 267696:  Memory - illegal accesses  (USE_AFTER_FREE)
>  >>>     Calling "memset" dereferences freed pointer "cur_path".
>  215                     memset(cur_path, 0, path_size * sizeof(char));
>  216                     free(cur_path);
>  217
>  218                 }
>  219
>  220                 //



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


Mime
View raw message