You’re passing a Null through your GetParameter nodes which most likely means you’re not setting the Length parameter - but that’s the whole point of your post…
I tried doing this yesterday afternoon and had the same results. The problem is that Length for a line based family is more like a reporting parameter than a true family parameter. I think you’d be better off placing a new family instance at the correct location/length rather than trying to modify the existing one.