Hello firepro20,

Unfortunately nothing was outputted on the screen.

A big part of the problem here is simply the layout, which makes it hard to see the structure of the code. With a bit of indentation:

$log = "/home/tsec/prototype/logs/extractedlogs/cowrieresult.log";

open(DATA, $log) or die "Can't open '$log': $!";

sub tester()
{
    while(<DATA>){
        ...
    }

    for my $ip (keys %HONEY){
        ...
    }

    for my $data (sort @DATA){
        ...
    }
}
[download]

See the problem now? sub tester is defined, but never called! You have to invoke the sub:

my $log = '/home/tsec/prototype/logs/extractedlogs/cowrieresult.log';
open(my $data, '<', $log) or die "Can't open '$log' for reading: $!";
tester($data);
close  $data              or die "Can't close '$log': $!";

sub tester
{
    my ($fh) = @_;

    while (<$fh>) {
        ...
    }
    ...
}
[download]

and then you should see the output you were expecting. Some notes:

• Prefer the 3-argument form of open (it’s clearer and safer).
• For filehandles, prefer lexical variables to barewords. Also, avoid using DATA for files, because this will confuse readers of your code: they will expect it to have its pre-defined meaning.
• Pass arguments to subroutines explictly via @_.
• Don’t use subroutine prototypes unless you have a good reason. In sub tester() the () is a prototype specifying that the sub takes no arguments.

Hope that helps,