There are several instances where ecalloc is called with swapped parameters (i.e. "size, number" instead of "number, size"). This makes no difference in practice since the values are multiplied, but we should still correct this.

Note: I only noticed this for ecalloc but we would do well to check other allocation functions as well.