Hello,

These patches remove empty strings and leading/trailing whitespace from DBus    
                                        
input for Domains.Configuration, Proxy.Configuration, Timeservers.Configuration 
                                        
and Nameservers.Configuration.                                                  
                                        
                                                                                
                                        
Previously, Nameservers.Configuration accepted invalid values such as           
                                        
"192.0.2.1  invalid-ip 8.8.8.8 " and then split them with space as delimiter,   
                                        
resulting in Nameservers.Configuration containing                               
                                        
[192.0.2.1, , invalid-ip, 8.8.8.8, ] rather than the valid value                
                                        
[192.0.2.1, 8.8.8.8].                                                           
                                        
                                                                                
                                        
The v2 implements remove_empty_strings much more sensibly, fixes some style     
                                        
issues and adds a patch for Nameservers.Configuration.    

Managed to fudge the cover letter first time around, sorry.

Jaakko Hannikainen (6):
  service: Add function to remove empty strings
  service: Remove empty strings from Domains
  service: Remove empty strings from Proxy
  service: Strip whitespace from Proxy URL
  service: Remove empty strings from Timeservers
  service: Remove empty strings from Nameservers

 src/service.c | 109 +++++++++++++++++++++++++++++++++++++++-------------------
 1 file changed, 74 insertions(+), 35 deletions(-)

-- 
2.1.0

_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to