Don't need to look check *input_name twice [CID-27].
This commit is contained in:
		| @@ -41,20 +41,20 @@ gss_release_name(OM_uint32 *minor_status, | |||||||
| 	    return GSS_S_COMPLETE; | 	    return GSS_S_COMPLETE; | ||||||
|  |  | ||||||
| 	name = (struct _gss_name *) *input_name; | 	name = (struct _gss_name *) *input_name; | ||||||
| 	if (name) { |  | ||||||
| 		if (name->gn_type.elements) | 	if (name->gn_type.elements) | ||||||
| 			free(name->gn_type.elements); | 		free(name->gn_type.elements); | ||||||
| 		while (SLIST_FIRST(&name->gn_mn)) { | 	while (SLIST_FIRST(&name->gn_mn)) { | ||||||
| 			struct _gss_mechanism_name *mn; | 		struct _gss_mechanism_name *mn; | ||||||
| 			mn = SLIST_FIRST(&name->gn_mn); | 		mn = SLIST_FIRST(&name->gn_mn); | ||||||
| 			SLIST_REMOVE_HEAD(&name->gn_mn, gmn_link); | 		SLIST_REMOVE_HEAD(&name->gn_mn, gmn_link); | ||||||
| 			mn->gmn_mech->gm_release_name(minor_status, | 		mn->gmn_mech->gm_release_name(minor_status, | ||||||
| 			    &mn->gmn_name); | 					      &mn->gmn_name); | ||||||
| 			free(mn); | 		free(mn); | ||||||
| 		} |  | ||||||
| 		gss_release_buffer(minor_status, &name->gn_value); |  | ||||||
| 		free(name); |  | ||||||
| 		*input_name = GSS_C_NO_NAME; |  | ||||||
| 	} | 	} | ||||||
|  | 	gss_release_buffer(minor_status, &name->gn_value); | ||||||
|  | 	free(name); | ||||||
|  | 	*input_name = GSS_C_NO_NAME; | ||||||
|  |  | ||||||
| 	return (GSS_S_COMPLETE); | 	return (GSS_S_COMPLETE); | ||||||
| } | } | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Love Hornquist Astrand
					Love Hornquist Astrand