Bug 122069

Summary: lokkit -p 53 core dumps
Product: [Fedora] Fedora Reporter: Tim Taiwanese Liim <tim.liim>
Component: redhat-config-securitylevelAssignee: Brent Fox <bfox>
Status: CLOSED DUPLICATE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 1   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://download.fedora.redhat.com/pub/fedora/linux/core/1/SRPMS/redhat-config-securitylevel-1.2.11-1.src.rpm
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-21 19:02:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tim Taiwanese Liim 2004-04-30 06:31:04 UTC
Description of problem:
  running "lokkit -p 53" core dumps; it should be able to take
  numeric port.  Problem found and fixed.  Please incorporate
  into source code.

Version-Release number of selected component (if applicable):
  redhat-config-securitylevel-1.2.11-1.src.rpm

How reproducible:
  always, on i386 platform.

Steps to Reproduce:
  1. as root, run "lokkit -p 53"
  
Actual results:
  core dump

Expected results:
  take additional port into Customier/Other Ports.

Additional info:
  how to fix:
   1. line 239:
	  char *ret;
	  struct servent *service;
       should be
     	  char *ret               = NULL;
	  struct servent *service = NULL;
       always good to init pointers to NULL.
   2. line 249
	ret = calloc((orig ? strlen(orig) : 0) + strlen(ret) + 2, sizeof(char));
      should be
	ret = calloc((orig ? strlen(orig) : 0) + strlen(tmp) + 2, sizeof(char));
      the core dump occurs at strlen(ret) as ret was not 
      initialized.

Comment 1 Brent Fox 2004-06-24 19:08:04 UTC

*** This bug has been marked as a duplicate of 117183 ***

Comment 2 Red Hat Bugzilla 2006-02-21 19:02:53 UTC
Changed to 'CLOSED' state since 'RESOLVED' has been deprecated.