 
            
            
            
            
                4 Nov
                
                    2016
                
            
            
                4 Nov
                
                '16
                
            
            
            
        
    
                6:22 p.m.
            
        On Thu, Nov 03, 2016 at 01:04:27AM +0800, mengdong.lin@linux.intel.com wrote:
- name = strlen(cfg->name) ? cfg->name : NULL;
- stream_name = strlen(cfg->stream_name) ? cfg->stream_name : NULL;
Please replace these with normal if statements. Also, are we sure that we've sufficiently audited the input paths to ensure that the strings are always terminated?