Calling Dispose() on the SPSite that the OpenWeb method was used on will automatically dispose of the SPWeb it opened. As long as the SPSite is being disposed of, you won't have issues here unless you are opening a ton of SPWeb's and not disposing them explicitly.

Looking at that code, it appears to be disposing of the SPSite, so it should be fine.

Calling Dispose() on the SPSite that the OpenWeb method was used on will automatically dispose of the SPWeb it opened. As long as the SPSite is being disposed of, you won't have issues here unless you are opening a ton of SPWeb's and not disposing them explicitly.

Looking at that code, it appears to be disposing of the SPSite, so it should be fine.

EDIT

For reference, here is the .Close() method from the SPSite class called by the .Dispose() method. As you can see, it disposed of any SPWebs it opened before disposing itself:

public void Close()
{
    SPEventManager.WaitForPostEvents();
    if (this.m_openedWebs != null)
    {
        SPWeakObjectHandleList<SPWeb> list = new SPWeakObjectHandleList<SPWeb>(this.m_openedWebs);
        foreach (SPWeb web in list)
        {
            web.Close();
        }
        list.Dispose();
        this.m_openedWebs.Dispose();
    }
    this.m_rootWebCreated = false;
    SPRequestContext.UnregisterSite(this);
    this.InvalidateSite();
}