Unfortunately, there are plotholes
Spoilers follow!
Specifically, in the climax. After the attack on Henry Baskerville, Stapleton follows Holmes back to the cage in which he kept the hound. When Holmes drops down into the cage(!?), Stapleton locks him in, returns to Baskerville Hall, and attempts to kill Henry again.
If Stapleton is intent on killing Sir Henry and claiming the estate, then he must be planning to kill Holmes as well. The problem is that, since he believes that Holmes is aware of his evil plans, it would be logical to assume that Watson is also informed of everything. Therefore, any further attempts on Sir Henry's life would be pointless.
Lastly, the omission of Holmes' request that Sir Henry walk rather the drive home that evening forces the viewer to conclude that Holmes is staking his elaborate efforts to bring the villain to justice in the mere chance that Sir Henry would walk home. In my opinion, these additions and omissions only hurt the film.