TCP RACK: don't log an uninitialized value

[tuexen]

Summary

reduce is uninitialized, if the code path for logging is reached via goto old_method;.
CID: 1557359

[cc]

Why not move the old_method: label above the stack variables' declaration? I think it may be cleaner to read.
Like this:

	old_method:
		uint64_t cwnd, tr_perms = 0;
		int32_t reduce = 0;

[tuexen]

In D48346#1103376, @cc wrote:

Why not move the old_method: label above the stack variables' declaration? I think it may be cleaner to read.
Like this:

	old_method:
		uint64_t cwnd, tr_perms = 0;
		int32_t reduce = 0;

I prefer declarations before code in a block and label inside the code block. Maybe this is just a personal preference. But I also had to look up the specification if Coverity is correct in stating that the variable might be unused...