Is this code safe?
——————————————-
function bid() public payable {
require(msg.value > lastPrice);

address payable previousOwner = payable(currentOwner);
previousOwner.send(msg.value);

currentOwner = msg.sender;
lastPrice = msg.value;
}
——————————————-
I dont want to use a separate withdrawal function, to save gas. But I also don't want to use transfer because the current owner could be a malicious contract that always fails when eth is sent to it, making it impossible for anyone to outbid him. Am I right that with this implementation if currentOwner is a malicious contract that will fail on eth send, the function would still execute regardless of send failing?