Re: [PATCH] sshd: fix typos

Orion Poplawski <[email protected]> Tue, 31 May 2016 10:41:19 -0600
Newsgroups gmane.comp.log.logwatch.devel
Message-ID <[email protected]>
On 05/30/2016 12:58 AM, Jan Synacek wrote:
> On Thu, May 19, 2016 at 8:39 PM, Markus Lude <[email protected]> wrote:
>> On Thu, May 19, 2016 at 02:05:21PM +0200, Jan Synacek wrote:
>>> ---
>>>  scripts/services/sshd | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/scripts/services/sshd b/scripts/services/sshd
>>> index a7edb41..430324f 100755
>>> --- a/scripts/services/sshd
>>> +++ b/scripts/services/sshd
>>> @@ -809,7 +809,7 @@ if (keys %ChownErr) {
>>>  }
>>>
>>>  if ( ($Detail == 7 && keys %Krb_relm > 1) || ($Detail > 8 && keys %Krb_relm) ){
>>> -  print "\nSucessfull Kerberos Authentication from ",(scalar keys %Krb_relm)," relm:\n";
>>> +  print "\nSuccessful Kerberos Authentication from ",(scalar keys %Krb_relm)," realm:\n";
>>>    foreach my $relm (keys %Krb_relm) {
>>>      if($Detail > 9){
>>>        print "   ",$relm,":\n";
>>
>> why not fix the typo in variable Krb_relm too?
> 
> For me, the variable name was not really important, but the output
> was. I can send version 2 of the patch that fixes the variable name as
> well if people are interested.
> 
> Cheers,
> 

Yeah, let's fix the variable name too.

-- 
Orion Poplawski
Technical Manager                     303-415-9701 x222
NWRA, Boulder/CoRA Office             FAX: 303-415-9702
3380 Mitchell Lane                       [email protected]
Boulder, CO 80301                   http://www.nwra.com

------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e